[gmime/gmime-2-4] fixed buffer overrun decoding 8bit headers



commit 3f224a21ef3798c3ca115cb62ec1ad2b1f6806c0
Author: Jeffrey Stedfast <fejj gnome org>
Date:   Tue Jan 12 11:08:16 2010 -0500

    fixed buffer overrun decoding 8bit headers
    
    2010-01-12  Jeffrey Stedfast  <fejj novell com>
    
    	* gmime/gmime-utils.c (g_mime_utils_decode_8bit): Made sure to
    	always increment inptr as we force-convert it to ascii. Thanks to
    	Damian Pietras for finding this bug.

 ChangeLog           |    6 ++++++
 gmime/gmime-utils.c |    4 +++-
 2 files changed, 9 insertions(+), 1 deletions(-)
---
diff --git a/ChangeLog b/ChangeLog
index bb96f14..c3e10f5 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2010-01-12  Jeffrey Stedfast  <fejj novell com>
+
+	* gmime/gmime-utils.c (g_mime_utils_decode_8bit): Made sure to
+	always increment inptr as we force-convert it to ascii. Thanks to
+	Damian Pietras for finding this bug.
+
 2009-11-30  Jeffrey Stedfast  <fejj novell com>
 
 	* configure.in: Don't include EXTRA_LIBS in GMIME_LIBS, instead
diff --git a/gmime/gmime-utils.c b/gmime/gmime-utils.c
index c1333d9..507462b 100644
--- a/gmime/gmime-utils.c
+++ b/gmime/gmime-utils.c
@@ -1670,9 +1670,11 @@ g_mime_utils_decode_8bit (const char *text, size_t len)
 		
 		while (inptr < inend) {
 			if (is_ascii (*inptr))
-				*outbuf++ = *inptr++;
+				*outbuf++ = *inptr;
 			else
 				*outbuf++ = '?';
+			
+			inptr++;
 		}
 		
 		*outbuf++ = '\0';



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]