Oscam absturz

[OBrecht]

dazu bräuchte er vielleicht ein Log - Level 4096 - aus einer Debug-Version zum Zeitpunkt Crash.

wobei:

uint32_t len = 0;
int8_t len = 0;​

wäre nicht uint8_t besser?

 

[Kind regards]

denke ich auch da wir kein Minus brauchen und uint8_t bis 270 geht (glaube ich gelesen zu haben)
und wobei es viel besser zu unsigned char passen würde.

 

[OBrecht]

Wertebereich:
+ int8_t >> 8 Bit >> -128..127 >> signed char
+ uint8_t >> 8 Bit >> 0..255 >> unsigned char

 

[TantäGiselar]

Bestätige das Problem mit 11420, auf 11410 runtergegangen und da ging es wieder. Ist mit schleierhaft woran es liegt, vor 6 Wochen hatte ich das Problem mit derselben Version noch nicht.

 

[pehedima]

Eig. dürfte die Deklaration der Längenvariablen 'len' nicht kleiner sein, als die max. mögliche Anzahl der zu schreibenden Bytes.
Diese wäre im o.a. Fall 1024.

D.h. aus Sicherheitsgründen müsste die Deklaration dann wie folgt aussehen:

[ u]int16_t len = 0;

Denke, @gorgone wird es selber viel besser wissen, wie er das zu korrigieren hat.

@TantäGiselar
Steht ja bereits in diesem Thread, warum es nicht geht. Einfach mal lesen.

 

[gorgone]

hmh ja negative werte ergeben in der tat keinen sinn an dieser position

Sie müssen registriert sein, um Links zu sehen.

mal sehen ob der neue gcc wieder meckert oder diesesmal der llvm/clang

 

[OBrecht]

    char emmdata[1025] = {'\0'}; /*1024 + '\0'*/
    unsigned char emmhex[513] = {'\0'};
    char buff[5] = {'\0'};
    uint8_t len = 0;
    cs_strncpy(emmdata, ep, sizeof(emmdata));
    remove_white_chars(emmdata);

    if('\0' != emmdata[0])
    {
      len = strlen(emmdata);

hier wäre vielleicht uint16_t besser wie pehedima schrieb. Buffer emmdata ist mit size 1025 angelegt, \0 weg als terminierendes Zeichen ergibt max 1024 (E-APDU).
Die Methode strlen(...) liefert size_t zurück.

 

[gorgone]

autsch du hast recht ich bin von 256 ausgegangen manchmal ist mann betriebsblind

Sie müssen registriert sein, um Links zu sehen.

 

[OBrecht]

wo sind die Tester:

• svn up
  
• kompilieren auf jeweiligem System
  
• Austausch alte oscam - Debug - durch V. 11423
• Debug-Level 4096 o. ALL
  
• Test mit EMM's die zum Crash geführt haben
• Ergebnnis bekanntgeben - SUCCESS/FAILURE

kannn es leider mangels Masse Karte, die diese Probleme verursacht, nicht selber testen.

 

[bl0w]

Also hab es mit der G09 und der C8 getestet (r11423) und keine Abstürze mehr...
Hab mal nur UQ getestet und noch kein S...

Also von mir ein Daumen hoch

 

[ghost0815]

Sobald ich Zeit habe teste und berichte ich.....

 

[FtW]

Passt nun durch r11421, hatte bis jetzt auch kein Crash mehr

 

[OBrecht]

r11421

Nimm lieber Minimum r11422.

 

[Derek Buegel]

.... oder gleich 11423.

MfG

 

[ghost0815]

So, habe 11423 getestet, bei mir kein Absturz mehr..
Danke @gorgone für fixen....