sprintf_s mit einem Puffer zu klein

Question

Der folgende Code verursacht einen Fehler und tötet meine Anwendung. Dies ist sinnvoll, da der Puffer nur 10 Byte lang ist und der Text 22 Byte lang ist (Pufferüberlauf).

Wie erhalte ich diesen Fehler, damit ich es melden kann, anstatt meine Anwendung zu stürzen?

Edit:

Nach dem Lesen der Kommentare unten ich mit _snprintf_s ging. Wenn es einen Wert -1 zurückgibt, wurde der Puffer nicht aktualisiert.

length = _snprintf_s(buffer, 10, 9, "123456789"); 
printf("1) Length=%d\n", length); // Length == 9 

length = _snprintf_s(buffer, 10, 9, "1234567890.1234567890."); 
printf("2) Length=%d\n", length); // Length == -1 

length = _snprintf_s(buffer, 10, 10, "1234567890.1234567890."); 
printf("3) Length=%d\n", length); // Crash, it needs room for the NULL char 

Answer 1

Anstelle von sprintf_s können Sie snprintf (a.k.a _snprintf unter Windows) verwenden.

#ifdef WIN32 
#define snprintf _snprintf 
#endif 

char buffer[10];  
int length = snprintf(buffer, 10, "1234567890.1234567890."); 
// unix snprintf returns length output would actually require; 
// windows _snprintf returns actual output length if output fits, else negative 
if (length >= sizeof(buffer) || length<0) 
{ 
    /* error handling */ 
} 

Answer 2

von MSDN:

Der andere Hauptunterschied zwischen sprintf_s und sprintf ist, dass eine Länge sprintf_s Parameter nimmt in Zeichen, die Größe des Ausgangspuffers spezifizieren. Wenn der Puffer für den zu druckenden Text zu klein ist, wird der Puffer auf eine leere Zeichenfolge gesetzt und der ungültige Parameterhandler wird aufgerufen. Im Gegensatz zu snprintf garantiert sprintf_s, dass der Puffer null-terminiert ist (es sei denn, die Puffergröße ist Null).

Also idealerweise sollte das, was Sie geschrieben haben, richtig funktionieren.

Answer 3

Sieht aus, als würdest du auf MSVC schreiben?

Ich denke, die MSDN-Dokumentation für sprintf_s sagt, dass es behauptet stirbt, so dass ich nicht sicher bin, ob Sie das programmatisch abfangen können.

Wie LBushkin vorgeschlagen hat, ist es viel besser, wenn Sie Klassen verwenden, die die Strings verwalten.

Answer 4

Es ist von Entwurf. Der gesamte Punkt von sprintf_s und anderen Funktionen aus der -Familie besteht darin, Buffer-Overrun-Fehler abzufangen und sie als Vorbedingungsverletzungen zu behandeln. Dies bedeutet, dass sie nicht wirklich wiederherstellbar sein sollen. Dies dient nur dazu, Fehler abzufangen - Sie sollten niemals sprintf_s aufrufen, wenn Sie wissen, dass die Zeichenfolge für einen Zielpuffer zu groß sein kann. Verwenden Sie in diesem Fall zuerst strlen, um zu prüfen, ob Sie eine Trimmung durchführen müssen.

Answer 5

Siehe Abschnitt 6.6.1 von TR24731, der die ISO C Committee Version der von Microsoft implementierten Funktionalität ist. Es bietet Funktionen set_constraint_handler(), abort_constraint_handler() und ignore_constraint_handler() Funktionen. Es gibt Kommentare von Pavel Minaev, die darauf hindeuten, dass die Microsoft-Implementierung nicht dem TR24731-Vorschlag entspricht (was ein "Typ-2-Tech-Bericht" ist), sodass Sie möglicherweise nicht eingreifen können oder etwas unternehmen müssen anders als das, was der TR angibt, sollte getan werden. Untersuchen Sie MSDN.

Answer 6

Dies funktioniert mit VC++ und ist sogar sicherer als snprintf mit (und sicherlich sicherer als _snprintf):

void TestString(const char* pEvil) 
{ 
    char buffer[100]; 
    _snprintf_s(buffer, _TRUNCATE, "Some data: %s\n", pEvil); 
} 

Die _TRUNCATE Flagge zeigt an, dass die Zeichenfolge abgeschnitten werden sollte. In dieser Form wird die Größe des Puffers nicht wirklich übergeben, was (paradoxerweise!) macht es so sicher. Der Compiler verwendet Vorlagenmagie, um die Puffergröße zu ermitteln, was bedeutet, dass sie nicht falsch angegeben werden kann (ein überraschend häufiger Fehler). Diese Technik kann angewendet werden, um andere sichere Zeichenfolgen-Wrapper zu erstellen, wie in meinem Blogpost hier beschrieben: https://randomascii.wordpress.com/2013/04/03/stop-using-strncpy-already/