snprintf Verwendung zu Pufferüberläufen

Question

gcc 4.4.1

C99

ich verwende snprintf wie dies zu vermeiden einen Pufferüberlauf zu vermeiden:

char err_msg[32] = {0}; 
snprintf(err_msg, sizeof(err_msg) - 1, "[ ST_ENGINE_FAILED ]"); 

ich die -1 hinzugefügt, um den Raum zu reservieren Null-Terminator für den Fall, dass die Zeichenfolge mehr als 32 Byte lang ist.

Bin ich richtig in meinem Denken?

Answer 1

Wie andere gesagt haben, brauchen Sie in diesem Fall nicht die -1. Wenn das Array eine feste Größe hat, würde ich stattdessen strncpy verwenden. Es wurde für das Kopieren von Zeichenfolgen erstellt - sprintf wurde für schwierige Formatierung erstellt. Wenn jedoch die Größe des Arrays unbekannt ist oder Sie versuchen zu ermitteln, wie viel Speicher für eine formatierte Zeichenfolge erforderlich ist. Dies ist, was ich wirklich wie über die Standard-Version von snprintf angegeben:

char* get_error_message(char const *msg) { 
    size_t needed = snprintf(NULL, 0, "%s: %s (%d)", msg, strerror(errno), errno); 
    char *buffer = malloc(needed+1); 
    sprintf(buffer, "%s: %s (%d)", msg, strerror(errno), errno); 
    return buffer; 
} 

Kombinieren Sie diese Funktion mit va_copy und Sie können sehr sicher formatiert String-Operationen erstellen. 

Answer 2

sizeof gibt die Anzahl der Bytes zurück, die der Datentyp im Speicher verwendet, und nicht die Länge der Zeichenfolge. Z.B. sizeof (int) gibt '4' Bytes auf einem 32-Bit-System zurück (gut, abhängig von der Implementierung, denke ich). Da Sie in Ihrem Array eine Konstante verwenden, können Sie diese problemlos an printf übergeben.

Answer 3

Sie brauchen nicht die -1, wie die Referenzzustände:

Die Funktionen snprintf() und vsnprintf() (einschließlich der Hinter ‚nicht mehr als Größe Bytes schreiben \ 0 ').

Beachten Sie die "einschließlich des nachgestellten '\ 0'" Teil

Answer 4

Nach snprintf(3):

Die Funktionen snprintf() und vsnprintf() nicht mehr als size Bytes schreiben (einschließlich der Hinter '\0') .

Answer 5

Keine Notwendigkeit für -1. C99 snprintfimmer nullterminiert. Size-Argument gibt die Größe des Ausgabepuffers einschließlich Zero Terminator an. Der Code, so wird

char err_msg[32]; 
int ret = snprintf(err_msg, sizeof err_msg, "[ ST_ENGINE_FAILED ]"); 

ret gedruckt tatsächliche Anzahl von Zeichen enthält (ohne Null Terminator).

jedoch nicht zu verwechseln mit Microsofts _snprintf (pre-C99), das tut nicht Null beenden, und, was das betrifft, hat völlig anderes Verhalten (zB Rückkehr -1 statt Möchtegern-Länge im Fall gedruckt wenn der Puffer nicht groß genug ist). Wenn Sie _snprintf verwenden, sollten Sie den gleichen Code wie in Ihrer Frage verwenden.

Answer 6

Für das Beispiel gegeben, sollten Sie diese stattdessen tun:

char err_msg[32]; 
strncpy(err_msg, "[ ST_ENGINE_FAILED ]", sizeof(err_msg)); 
err_msg[sizeof(err_msg) - 1] = '\0'; 

oder noch besser:

char err_msg[32] = "[ ST_ENGINE_FAILED ]";