(xstrdup): Oops, obviously I calculated the length wrong when
authorWerner Koch <wk@gnupg.org>
Mon, 1 Jul 2002 13:09:18 +0000 (13:09 +0000)
committerWerner Koch <wk@gnupg.org>
Mon, 1 Jul 2002 13:09:18 +0000 (13:09 +0000)
coded this.  Tsss, wrote xstrdup some hundreds times but missed it
this time.  Thanks to Steffen Hansen for noticing it.

gpgmeplug/ChangeLog
gpgmeplug/gpgmeplug.c

index 0572044..ae803c3 100644 (file)
        malloc. Removed superfluous string termination.
        (parseAddress): Use xmalloc instead of an unchecked malloc.
        (nextAddress): Ditto.
+       (xstrdup): Oops, obviously I calculated the length wrong when
+       coded this.  Tsss, wrote xstrdup some hundreds times but missed it
+       this time.  Thanks to Steffen Hansen for noticing it.
+
        * gpgmeplug.c:  Moved a few helper functions more to the top.
        Fixed comment syntax.  Merged a copyright notice somewhere in the
        middle of the file with the one at the top.
index 7a2ea25..0b039a4 100644 (file)
@@ -239,7 +239,7 @@ xmalloc (size_t n)
 static char *
 xstrdup (const char *string)
 {
-  char *p = xmalloc (strlen (string));
+  char *p = xmalloc (strlen (string)+1);
   strcpy (p, string);
   return p;
 }