Package: wget
Version: 1.12-2

http_atotm() does not copy the terminating NUL character of the locale
string.  The following patch should fix this.

(l < sizeof savedlocale holds at this point, so l is at most 254, and
there is room for NUL character, if I'm not mistaken.)

It's probably easier to just use xstrdup, by the way.

commit 1182dcd947bc5718af6a1c797a79487607fcfc9f
Author: Florian Weimer <[email protected]>
Date:   Sun May 16 11:57:52 2010 +0200

    http_atotm(): also copy terminating NUL character

diff --git a/src/http.c b/src/http.c
index 15e1caf..e7dc830 100644
--- a/src/http.c
+++ b/src/http.c
@@ -2928,7 +2928,7 @@ http_atotm (const char *time_string)
       if (l >= sizeof savedlocale)
         savedlocale[0] = '\0';
       else
-        memcpy (savedlocale, oldlocale, l);
+        memcpy (savedlocale, oldlocale, l + 1);
     }
   else savedlocale[0] = '\0';
 



-- 
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]

Reply via email to