C: Cppcheck: Mögliche Nullpunktdereferenz

Question

Also hier in main.c, habe ich diesen Teil des Codes, der den Inhalt von verschlüsselten druckt, wenn es nicht leer ist. Es ist wirklich so einfach.

CPP Fehler ist:

[main.c: 40]: (Fehler) Mögliche Nullzeiger dereferenzieren: verschlüsselten - sonst ist es überflüssig, wenn an der Leitung 31 ist

null verschlüsselt zu überprüfen

Der Code:

char* encrypted = bmp_encrypt(key, text); 
    if(encrypted != NULL) //error points here (line 31) 
    { 
     printf("Encrypted:"); 
     for(int i=0; i<strlen(text);i++) 
     { 
      printf("%x ", (unsigned char) encrypted[i]); 
     } 
     printf("\n"); 
    } 
    else{printf("Encrypted:%s\n", encrypted);} //this is line 40 

Das Ding ist, seine Arbeits soll als aber cppcheck hält mich nervt, sollte ich es beheben? ist es falsch das zu tun?

Answer 1

Der else Block Ihres Codes wird nur eingegeben, wenn encrypted NULL ist. Sie übergeben also einen Nullzeiger an printf. Das kann undefined behavior aufrufen.

Da Sie wissen, dass der Zeiger an diesem Punkt NULL ist, drucken Sie nur ausdrücklich, dass es NULL ist:

else{printf("Encrypted: (null)\n");}