From: Nigel Metheringham Date: Fri, 12 Dec 2008 14:36:37 +0000 (+0000) Subject: Buffer overrun fix. fixes: bug #787 X-Git-Tag: DEVEL_PDKIM_START~13 X-Git-Url: https://git.exim.org/exim.git/commitdiff_plain/24c929a2 Buffer overrun fix. fixes: bug #787 --- diff --git a/doc/doc-txt/ChangeLog b/doc/doc-txt/ChangeLog index 52279d332..09b9031cc 100644 --- a/doc/doc-txt/ChangeLog +++ b/doc/doc-txt/ChangeLog @@ -1,4 +1,4 @@ -$Cambridge: exim/doc/doc-txt/ChangeLog,v 1.556 2008/10/16 07:57:01 nm4 Exp $ +$Cambridge: exim/doc/doc-txt/ChangeLog,v 1.557 2008/12/12 14:36:37 nm4 Exp $ Change log file for Exim from version 4.21 ------------------------------------------- @@ -77,6 +77,9 @@ NM/07 Bugzilla 769: Extraneous comma in usage fprintf NM/08 Fixed erroneous documentation references to smtp_notquit_acl to be acl_smtp_notquit +NM/09 Bugzilla 787: Potential buffer overflow in string_format + Patch provided by Eugene Bujak + Exim version 4.69 ----------------- diff --git a/src/src/string.c b/src/src/string.c index 20bd1d1f3..83455294d 100644 --- a/src/src/string.c +++ b/src/src/string.c @@ -1,4 +1,4 @@ -/* $Cambridge: exim/src/src/string.c,v 1.13 2007/02/26 14:07:04 ph10 Exp $ */ +/* $Cambridge: exim/src/src/string.c,v 1.14 2008/12/12 14:36:37 nm4 Exp $ */ /************************************************* * Exim - an Internet mail transport agent * @@ -1267,10 +1267,17 @@ while (*fp != 0) not OK, add part of the string (debugging uses this to show as much as possible). */ + if (p == last) + { + yield = FALSE; + goto END_FORMAT; + } if (p >= last - width) { yield = FALSE; width = precision = last - p - 1; + if (width < 0) width = 0; + if (precision < 0) precision = 0; } sprintf(CS p, "%*.*s", width, precision, s); if (fp[-1] == 'S')