[PATCH] Fix buffer overflow caused by printf implementation not subtracting space used from space available.

Andrew Miller ak.miller at auckland.ac.nz
Tue May 31 21:17:23 CDT 2011


Fixing Bug 27348. I wrote a simple test program that works on Windows 
but crashes on Wine without this patch (and works on Wine with this 
patch applied); the program is given as an attachment on the Bugzilla at 
winehq.org (http://bugs.winehq.org/show_bug.cgi?id=27348).

---
  dlls/msvcrt/printf.h |    1 +
  1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/dlls/msvcrt/printf.h b/dlls/msvcrt/printf.h
index a69e8f8..eefecd1 100644
--- a/dlls/msvcrt/printf.h
+++ b/dlls/msvcrt/printf.h
@@ -56,6 +56,7 @@ static int FUNC_NAME(puts_clbk_str)(void *ctx, int 
len, const APICHAR *str)

      memcpy(out->buf, str, len*sizeof(APICHAR));
      out->buf += len;
+    out->len -= len;
      return len;
  }

-- 

Please note: I don't have time to follow this patch through the process 
or make any further changes that may be required - I would greatly 
appreciate it if someone else could adopt the patch. I won't be on the 
wine-patches e-mail list for very long, so please keep me CCd if you 
want to contact me.

This bug causes a crash with libxml2 programs on Wine if libxml2 has to 
produce an error message (which uses _vsnprintf) and so could 
potentially be affecting a number of programs.

Best wishes,
Andrew



More information about the wine-patches mailing list