Ticket #8409: vsnprintf.diff

backends/gp32/debug-gdbstub-usb.cpp

@@ -994 +994 @@
         if (!g_GDBConnected) return;
         
         va_start(VaList , pFormat);
-        vsprintf(Temp , pFormat , VaList);
+        vsnprintf(Temp, 0x100, pFormat , VaList);
         va_end(VaList);
 
         iLength = strlen(Temp);

backends/gp32/gp32.cpp

@@ -2430 +2430 @@
                 return 0;
         busy = true;
         va_start(marker, fmt);
-        vsprintf(s, fmt, marker);
+        vsnprintf(s, 1024, fmt, marker);
         va_end(marker);
 
 #ifdef GPDEBUG
@@ -2460 +2460 @@
         va_list marker;
 
         va_start(marker, fmt);
-        vsprintf(s, fmt, marker);
+        vsnprintf(s, 256, fmt, marker);
         va_end(marker);
 
         return fwrite(s, 1, strlen(s), stream);

backends/wince/missing/missing.cpp

@@ -567 +567 @@
         va_list va;
 
         va_start(va, format);
-        vsprintf(buf, format, va);
+        vsnprintf(buf, 1024, format, va);
         va_end(va);
 
         if (buf[strlen(buf) - 1] == '\n') {

base/engine.cpp

@@ -92 +92 @@
         va_list va;
 
         va_start(va, s);
-        vsprintf(buf_input, s, va);
+        vsnprintf(buf_input, STRINGBUFLEN, s, va);
         va_end(va);
 
         if (g_engine) {

scumm/debugger.cpp

@@ -51 +51 @@
                 return;
 
         va_start(va, s);
-        vsprintf(buf, s, va);
+        vsnprintf(buf, STRINGBUFLEN, s, va);
         va_end(va);
 
         debug(buf);

scumm/scumm.cpp

@@ -2779 +2779 @@
         va_list va;
 
         va_start(va, message);
-        vsprintf(buf, message, va);
+        vsnprintf(buf, STRINGBUFLEN, message, va);
         va_end(va);
 
         GUI::MessageDialog dialog(buf, "OK", altButton);

simon/charset.cpp

@@ -223 +223 @@
         va_list va;
 
         va_start(va, s);
-        vsprintf(buf, s, va);
+        vsnprintf(buf, STRINGBUFLEN, s, va);
         va_end(va);
 
         if (!_fcs_data_1[_fcs_unk_1]) {

sky/control.cpp

@@ -1052 +1052 @@
 }
 
 int Control::displayMessage(const char *altButton, const char *message, ...) {
-#ifdef __PALM_OS__
-        char buf[256]; // 1024 is too big overflow the stack
-#else
-        char buf[1024];
-#endif
+        char buf[STRINGBUFLEN];
         va_list va;
 
         va_start(va, message);
-        vsprintf(buf, message, va);
+        vsnprintf(buf, STRINGBUFLEN, message, va);
         va_end(va);
 
         GUI::MessageDialog dialog(buf, "OK", altButton);

sound/softsynth/mt32.cpp

@@ -174 +174 @@
 static void MT32_PrintDebug(void *userData, const char *fmt, va_list list) {
         char buf[512];
         if (((MidiDriver_MT32 *)userData)->_initialising) {
-                vsprintf(buf, fmt, list);
+                vsnprintf(buf, 512, fmt, list);
                 buf[70] = 0; // Truncate to a reasonable length
                 drawMessage(1, buf);
         }

sword1/control.cpp

@@ -702 +702 @@
         va_list va;
 
         va_start(va, message);
-        vsprintf(buf, message, va);
+        vsnprintf(buf, STRINGBUFLEN, message, va);
         va_end(va);
 
         GUI::MessageDialog dialog(buf, "OK", altButton);

tools/convbdf.c

@@ -101 +101 @@
         va_list va;
 
         va_start(va, s);
-        vsprintf(buf, s, va);
+        vsnprintf(buf, 1024, s, va);
         va_end(va);
 
         fprintf(stderr, "ERROR: %s!\n", buf);
@@ -114 +114 @@
         va_list va;
 
         va_start(va, s);
-        vsprintf(buf, s, va);
+        vsnprintf(buf, 1024, s, va);
         va_end(va);
 
         fprintf(stderr, "WARNING: %s!\n", buf);

tools/md5table.c

@@ -31 +31 @@
         va_list va;
 
         va_start(va, s);
-        vsprintf(buf, s, va);
+        vsnprintf(buf, 1024, s, va);
         va_end(va);
 
         fprintf(stderr, "ERROR: %s!\n", buf);
@@ -44 +44 @@
         va_list va;
 
         va_start(va, s);
-        vsprintf(buf, s, va);
+        vsnprintf(buf, 1024, s, va);
         va_end(va);
 
         fprintf(stderr, "WARNING: %s!\n", buf);