[C] Char Array an Funktion übergeben

[Jan H. (jhnet)]

Moin,

ich habe eine Methode die mit einem String aufgerufen werden soll, dann 
eine Checksume dranhängt und das ganze per UART von PeterFleury 
versendet.
Das sieht so aus:

static char calculate_checksum(char data[]) {
    char checksum = 0x00;
    for (uint8_t i = 0; i < strlen(data); i++) {
        checksum ^= data[i];
    }
    return checksum;
}
void send_uart_message(char message[]) {
    char checksum = calculate_checksum(message);
    char buffer[100];
    strcpy(buffer, "$");
    strcat(buffer, message);
    strcat(buffer, "*");
    char checksum_string[2];
    sprintf(checksum_string, "%02X", checksum);
    strcat(buffer, checksum_string);
    strcat(buffer, "\r\n");
    uart_puts(buffer);
}

Das ganze funktioniert nun, wenn ich es mit

send_uart_message("ABC");

aufrufe, ich möchte aber eigentlich soetwas machen, was aber zu einem 
Dauer-Reset führt:

char message[120];
sprintf(message, "CMP,%d,%d,%d,%d,%d,%d,%d,%d,%d", compass.yaw, compass.pitch,compass.roll, compass.acc_x, compass.acc_y, compass.acc_z,compass.mag_x,compass.mag_y,compass.mag_z);
send_uart_message(message);

Wie kann ich das Problem beheben ?
Danke im Voraus !

[Peter II (Gast)]

Jan H. schrieb:
> char checksum_string[2];
> sprintf(checksum_string, "%02X", checksum);

dein Checksum_string ist zu kurz, checksum kann ja scheinbar bis 255 
werden. Dazu kommt noch das 0 Byte. Also sollte er mindestens 4byte lang 
sein.


> for (uint8_t i = 0; i < strlen(data); i++) {
ist etwas ungünstig, im Zweifelsfall wird stündig strlen aufgerufen.

for( uint8_t i = 0; data[i]; i++) {

[Rufus Τ. F. (rufus)]

Peter II schrieb:
> dein Checksum_string ist zu kurz, checksum kann ja scheinbar bis 255
> werden. Dazu kommt noch das 0 Byte. Also sollte er mindestens 4byte lang
> sein.

Drei reichen bei hexadezimaler Darstellung. Aber drei müssen es schon 
sein.

[Peter II (Gast)]

ausßerdem ist ist überhaupt nicht notwendig den string komplett 
zusammenzubauen, kostet nur zeit uns stack.

void send_uart_message(char message[]) {
    char checksum = calculate_checksum(message);
    char checksum_string[3];
    sprintf(checksum_string, "%02X", checksum);

    uart_puts("$");
    uart_puts("message");
    uart_puts("*");
    uart_puts( checksum_string );
    uart_puts( "\r\n");
}

sollte das gleiche machen, aber etwas weniger speicher brauchen.

[Jan H. (jhnet)]

Jo danke nun funzt es, an "\0" hätte ich ja mal denken können -.-
Wundert mich nur, dass es mit "ABC" inkl. Checksumme funktioniert hat.


Peter II schrieb:
> ausßerdem ist ist überhaupt nicht notwendig den string komplett
> zusammenzubauen, kostet nur zeit uns stack.

> sollte das gleiche machen, aber etwas weniger speicher brauchen.


Jo, wo du es sagst...Warum eigentlich nicht !

[Karl H. (kbuchegg)]

Und die Variable i in der Checksum Funktion braucht auch keiner

static char calculate_checksum(char data[]) {
    char checksum = 0x00;

    while( *data )
      checksum ^= *data++;
    }

    return checksum;
}

womit du ganz nebenbei die Beschränkung auf Strings der Maximallänge von 
255 Character losgeworden bist.

[Karl H. (kbuchegg)]

Jan H. schrieb:

> Wundert mich nur, dass es mit "ABC" inkl. Checksumme funktioniert hat.

Das dumme an undefiniertem Verhalten ist, das alles mögliche passieren 
kann. Inklusive dem Verhalten, welches du erwartet hast.
Das bedeutet allerdings nicht, dass es deshalb in Ordnung wäre. 
Fehlerhaft ist fehlerhaft, selbst wenn zufällig das richtige rauskommt, 
weil du zufälig gerade irgendein momentan nicht benutztes Byte im 
Speicher niedergebügelt hast. Bei der nächsten Programmänderung oder 
-erweiterung kann das alles schon wieder ganz anders aussehen.