sprintf feuer ISR

[Martin S. (sirnails)]

Guten Tag,

ich arbeite gerade an einem M3. Ich habe folgenden Code, der mir jedes 
Mal beim sprintf einen Fault-Interrupt feuert:

void
DebugShowCoordinates(signed int sx, signed int sy) {
  char OutBuffer[30] = {0};

  sprintf(OutBuffer, "FROM %i TO %i", sx, sy);
  SetErrorMessage(OutBuffer, 3, true);
};

Wo liegt da der Unterschied zum funktionierenden Code:

char txt[20];
unsigned short PixelBuffer[0];

retval = EdgeDetection(120, true, 320, 240);
CameraImageDataGet(false, (NEWPOS.x * 2), 120, 1, false, PixelBuffer);

sprintf(txt, "x=%i, y=%i, BRGHT=%u", retval.x, retval.y, (BLUEFROM565(PixelBuffer[0]) + GREENFROM565(PixelBuffer[0]) + REDFROM565(PixelBuffer[0])) / 3);
SetErrorMessage(txt, 1, true);

Hab ich Tomaten auf den Augen? Ich sehe da keinen Unterschied?!

Danke, Grüße
M. Schwaikert

[Rufus Τ. F. (rufus)]

Verwende sicherheitshalber snprintf, auch wenn es eher 
unwahrscheinlich ist, daß Dein 30 Zeichen großer Puffer überschrieben 
wird.

Hast Du denn noch genügend Platz auf dem Stack? Als automatische 
Variable landet OutputBuffer natürlich dort.

Ansonsten:

  signed int sx

Das signed hier ist völlig überflüssig.

Anders als bei der heiligen Dreifaltigkeit char, unsigned char und 
signed char gibt es nur zwei Varianten beim int :

int und unsigned int.

[Martin S. (sirnails)]

Rufus Τ. Firefly schrieb:
> Verwende sicherheitshalber snprintf, auch wenn es eher
> unwahrscheinlich ist, daß Dein 30 Zeichen großer Puffer überschrieben
> wird.

Mach ich mal - nope, fliegt auch auseinander.

> Hast Du denn noch genügend Platz auf dem Stack? Als automatische
> Variable landet OutputBuffer natürlich dort.

Ja, der ist 2048 Byte groß. Das sollte reichen :)

>
> Ansonsten:
>
>   signed int sx
>
> Das signed hier ist völlig überflüssig.

Ich weiß. Es war nur zur Verdeutlichung, dass es sich eben um einen 
Signed handelt, und ich das nicht beim groben drüberfliegen übersehe.

[Martin S. (sirnails)]

Hmm... also irgendwas überschreibt mir die Rücksprungadresse. Selbst 
wenn ich das snprinft auskommentiere, so zerlegt es mir das Programm 
beim SetErrorMessage.

EDIT: Kommando zurück, ich hab tatsächlich 2kB Stack verbraten. Hab ihn 
jetzt auf 4kB vergrößert und der Fehler ist weg.

Danke :)