From: Werner Koch <wk@gnupg.org>
Date: Mon, 1 Jul 2002 13:09:18 +0000 (+0000)
Subject: (xstrdup): Oops, obviously I calculated the length wrong when
X-Git-Tag: dd9jn_pre_test_20020702~4
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=ca43df065f8f1605b2ac32fa8635c25082f27bcf;p=gpgme.git

(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 0572044..ae803c3 100644
--- 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 7a2ea25..0b039a4 100644
--- 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;
 }