Prevent an off-by-one in a allocated buffer in glob_match_url() - detected by

coverity.com

diff --git a/src/urlglob.c b/src/urlglob.c
index 711a7e9f5204e01d56d6732e6a14fb69c8890e9b..b133fb0e8b565dafcc300871e75bc71c42f08c9b 100644 (file)
--- a/src/urlglob.c
+++ b/src/urlglob.c
@@ -496,7 +496,8 @@ char *glob_match_url(char *filename, URLGlob *glob)
    * be longer than the URL we use. We allocate a good start size, then
    * we need to realloc in case of need.
    */
-  allocsize=strlen(filename);
+  allocsize=strlen(filename)+1; /* make it at least one byte to store the
+                                   trailing zero */
   target = malloc(allocsize);
   if(NULL == target)
     return NULL; /* major failure */
@@ -548,7 +549,9 @@ char *glob_match_url(char *filename, URLGlob *glob)
     }
     if(appendlen + stringlen >= allocsize) {
       char *newstr;
-      allocsize = (appendlen + stringlen)*2;
+      /* we append a single byte to allow for the trailing byte to be appended
+         at the end of this function outside the while() loop */
+      allocsize = (appendlen + stringlen)*2 + 1;
       newstr=realloc(target, allocsize);
       if(NULL ==newstr) {
         free(target);