(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.

diff --git a/gpgmeplug/ChangeLog b/gpgmeplug/ChangeLog
index 0572044969f48669d9402d34430a5a8d43fcb3b6..ae803c3349a0d1d5e3a86e011f721a3025c85749 100644 (file)
--- a/gpgmeplug/ChangeLog
+++ b/gpgmeplug/ChangeLog
@@ -14,6 +14,10 @@
        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.

diff --git a/gpgmeplug/gpgmeplug.c b/gpgmeplug/gpgmeplug.c
index 7a2ea259b1f9550c2f9af8b6263fadcf70cced6c..0b039a418ffff39702c4285a26f4b55fde3ad961 100644 (file)
--- a/gpgmeplug/gpgmeplug.c
+++ b/gpgmeplug/gpgmeplug.c
@@ -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;
 }