Substituierter Code schmiert ab

[Progger (Gast)]

Hallo,

ich habe ein merkwürdiges Absturzverhalten:

Gegeben ist eine Funktion in der der folgende Code enthalten ist:

Funktion1()
{
...diverser Code...

uint8_t array[6] = {0};
sscanf(Struktur.Eintrag_eins, "%X:%X:%X:%X:%X:%X", (uint *) &array[0], (uint *) &array[1], (uint *) &array[2], (uint *) &array[3], (uint *) &array[4], (uint *) &array[5]);

for (uint8_t i = 0; i < 6; i++)
{
 Struktur.Eintrag_zwei[i] = array[i];
}
}

Nun kommt der detaillierte Code an weiteren Stellen auch vor und daher 
habe ich ihn in eine Funktion gepackt, d.h.

void Funktion2(void)
{
uint8_t array[6] = {0};
sscanf(Struktur.Eintrag_eins, "%X:%X:%X:%X:%X:%X", (uint *) &array[0], (uint *) &array[1], (uint *) &array[2], (uint *) &array[3], (uint *) &array[4], (uint *) &array[5]);

for (uint8_t i = 0; i < 6; i++)
{
 Struktur.Eintrag_zwei[i] = array[i];
}

}

und folgend:

Funktion1()
{
...diverser Code...
Funktion2();
}

Ich möchte also eigentlich nur substituieren, weil das an mehreren 
Stellen vorkommt, doch nun führt das plötzlich zu Abstürzen.

Bei der Struktur handelt es sich um eine globale Variable.


Jemand eine Idee, warum das so nicht funktioniert?

[abacus (Gast)]

Sieht für mich so aus, als hättest du wirr irgendwelche Pointer 
zusammengewürfelt bis der Compiler keine Warnungen mehr erzeugt hat.

[fop (Gast)]

Sieht so aus, als ob scanf mehr als ein Byte pro Wert in den Speicher 
schreibt. Der Fehler ist also schon im alten Code vorhanden, aber da hat 
es nichts sonderlich Wichtiges zerrissen.

Also muss die Definition

uint8_t array[6] = {0};

 angepasst werden.
Am Besten so, dass Typecasts wie Dein "(uint *)" nicht mehr nötig sind.

Bzw. mach die Typecasts weg und lass Dir die Compilerwarnungen auf der 
Zunge zergehen ;-)

[Nop (Gast)]

Wenn man unsigned ints, die mindestens zwei Byte breit ist, in ein Array 
aus uint8_t liest, kann es schonmal zu Problemen kommen. Daß es das 
vorher nicht tat, kann Zufall gewesen sein, weil dann halt beim Zugriff 
auf das letzte Array-Element irgendwas Unwesentliches überschrieben 
wurde.

Ändere den Datentyp bei dem Array mal auf unsigned int.

[Progger (Gast)]

Tatsache, der Fehler war schon immer da, hat sich nur nicht bemerkbar 
gemacht.

Danke!