Infinite Loop STM32 Zeichen ersetzen

Hast du vielleicht vergessen, die Zeichenkette vor dem Aufruf der 
Funktion mit 0 zu terminieren?

Peter schrieb:
> ich bekomme bei folgendem Code ein infinite Loop.

Zwei Moeglichkeiten:

a) Worauf pStr zeigt, ist kein C-String, i.e. nicht durch NUL terminiert
b) Du kaputtest diesen String irgendwo spaeter

HTH
leo

Ja, aber wie gesagt: Bereits nach fünf vergleichen geht der in infinite 
loop. Der String ist aber 0-terminiert und deutlich länger als fünf 
Zeichen.

Unter welchen Bedingungen tritt dieser Fehler eigentlich ein?

leo schrieb:
> Peter schrieb:
>> ich bekomme bei folgendem Code ein infinite Loop.
>
> Zwei Moeglichkeiten:
>
> a) Worauf pStr zeigt, ist kein C-String, i.e. nicht durch NUL terminiert
> b) Du kaputtest diesen String irgendwo spaeter

c) "replace" ist NUL, i.e. du vernichtest dein Stringende.

>
> HTH
> leo

Mein Aufruf ist der folgende:

1

strrpl(str, '\r', ' ');

Peter schrieb:
> Mein Aufruf ist der folgende:strrpl(str, '\r', ' ');

Ja dann musst du halt debuggen. Aus diesen Infos ist kein Fehler 
errsichtlich.

lei

Jim M. schrieb:
> Steht der String zufällig im Flash statt im RAM (weil Konstante)?

Danke, das war es!

Peter schrieb:
> Jim M. schrieb:
>> Steht der String zufällig im Flash statt im RAM (weil Konstante)?
>
> Danke, das war es!

"Nach einer
Hand voll Schleifendurchläufen bekomme ich den Fehler."

Das passt aber nicht ganz zusammen. Oder (Hand-1) Durchlaeufe waren 
NOPs.

leo

leo schrieb:
> Das passt aber nicht ganz zusammen. Oder (Hand-1) Durchlaeufe waren
> NOPs.

Doch, denn das fünfte Zeichen war ein '\r' :). Das wurde versucht zu 
ersetzen und Fehler!

Peter schrieb:
> static void strrpl(char *pStr, char replace, char newChar){
>   do {
>     if(*pStr == replace) *pStr = newChar;
>     pStr++;
>   } while(*pStr != '\0');
> }


ich würde diese do-while-Schleife durch eine while-Schleife ersetzen.

Bei dir wird immer die Adresse um eine Stelle verschoben. Erst dann wird 
geprüft.

Wenn es passiert dass die Funktion einen leeren String bekommt, dann 
wird solange geprüft bis im Speicher eine Null gefunden wird. Zumindest 
auf einem PC.

Peter schrieb:
> leo schrieb:
>> Das passt aber nicht ganz zusammen. Oder (Hand-1) Durchlaeufe waren
>> NOPs.
>
> Doch, denn das fünfte Zeichen war ein '\r' :). Das wurde versucht zu
> ersetzen und Fehler!

Ah. Die wichtigste Info im letzten Posting, nach Nachfragen. Super.

leo