Subversion Repositories

?revision_form?Rev ?revision_input??revision_submit??revision_endform?

Rev 32 | Go to most recent revision | Blame | Compare with Previous | Last modification | View Log | RSS feed

Author: Svante Signell <svante.signell@telia.com>
Author: Petter Reinholdtsen <pere@hungry.com>
Author: Magnus Holmgren <magnus@debian.org>
Bug-Debian: http://bugs.debian.org/657116
Description: Fix FTBFS on Hurd by dynamically allocating path names.
 Depends on no_static_buffers.patch, which introduced the th_pathname field.

--- a/compat/basename.c
+++ b/compat/basename.c
@@ -34,13 +34,25 @@ static char rcsid[] = "$OpenBSD: basenam
 #include <errno.h>
 #include <string.h>
 #include <sys/param.h>
+#include <stdlib.h>
 
 char *
 openbsd_basename(path)
        const char *path;
 {
-       static char bname[MAXPATHLEN];
+       static char *bname = NULL;
+       static size_t allocated = 0;
        register const char *endp, *startp;
+       int len = 0;
+
+       if (!allocated) {
+               allocated = 64;
+               bname = malloc(allocated);
+               if (!bname) {
+                       allocated = 0;
+                       return NULL;
+               }
+       }
 
        /* Empty or NULL string gets treated as "." */
        if (path == NULL || *path == '\0') {
@@ -64,11 +76,19 @@ openbsd_basename(path)
        while (startp > path && *(startp - 1) != '/')
                startp--;
 
-       if (endp - startp + 1 > sizeof(bname)) {
-               errno = ENAMETOOLONG;
-               return(NULL);
+       len = endp - startp + 1;
+
+       if (len + 1 > allocated) {
+               size_t new_allocated = 2*(len+1);
+               void *new_bname = malloc(new_allocated);
+               if (!new_bname)
+                       return NULL;
+               allocated = new_allocated;
+               free(bname);
+               bname = new_bname;
        }
-       (void)strncpy(bname, startp, endp - startp + 1);
-       bname[endp - startp + 1] = '\0';
+
+       (void)strncpy(bname, startp, len);
+       bname[len] = '\0';
        return(bname);
 }
--- a/compat/dirname.c
+++ b/compat/dirname.c
@@ -34,13 +34,25 @@ static char rcsid[] = "$OpenBSD: dirname
 #include <errno.h>
 #include <string.h>
 #include <sys/param.h>
+#include <stdlib.h>
 
 char *
 openbsd_dirname(path)
        const char *path;
 {
-       static char bname[MAXPATHLEN];
+       static char *bname = NULL;
+       static size_t allocated = 0;
        register const char *endp;
+       int len;
+
+       if (!allocated) {
+               allocated = 64;
+               bname = malloc(allocated);
+               if (!bname) {
+                       allocated = 0;
+                       return NULL;
+               }
+       }
 
        /* Empty or NULL string gets treated as "." */
        if (path == NULL || *path == '\0') {
--- a/lib/append.c
+++ b/lib/append.c
@@ -38,7 +38,7 @@ typedef struct tar_dev tar_dev_t;
 struct tar_ino
 {
        ino_t ti_ino;
-       char ti_name[MAXPATHLEN];
+       char ti_name[];
 };
 typedef struct tar_ino tar_ino_t;
 
@@ -61,7 +61,7 @@ tar_append_file(TAR *t, const char *real
        libtar_hashptr_t hp;
        tar_dev_t *td = NULL;
        tar_ino_t *ti = NULL;
-       char path[MAXPATHLEN];
+       char *path = NULL;
 
 #ifdef DEBUG
        printf("==> tar_append_file(TAR=0x%lx (\"%s\"), realname=\"%s\", "
@@ -126,34 +126,39 @@ tar_append_file(TAR *t, const char *real
        }
        else
        {
+               const char *name;
 #ifdef DEBUG
                printf("+++ adding entry: device (0x%lx,0x%lx), inode %ld "
                       "(\"%s\")...\n", major(s.st_dev), minor(s.st_dev),
                       s.st_ino, realname);
 #endif
-               ti = (tar_ino_t *)calloc(1, sizeof(tar_ino_t));
+               name = savename ? savename : realname;
+               ti = (tar_ino_t *)calloc(1, sizeof(tar_ino_t) + strlen(name) + 1);
                if (ti == NULL)
                        return -1;
                ti->ti_ino = s.st_ino;
-               snprintf(ti->ti_name, sizeof(ti->ti_name), "%s",
-                        savename ? savename : realname);
+               snprintf(ti->ti_name, strlen(name) + 1, "%s", name);
                libtar_hash_add(td->td_h, ti);
        }
 
        /* check if it's a symlink */
        if (TH_ISSYM(t))
        {
-               i = readlink(realname, path, sizeof(path));
+               if ((path = malloc(s.st_size + 1)) == NULL)
+                       return -1;
+               i = readlink(realname, path, s.st_size);
                if (i == -1)
+               {
+                       free(path);
                        return -1;
-               if (i >= MAXPATHLEN)
-                       i = MAXPATHLEN - 1;
+               }
                path[i] = '\0';
 #ifdef DEBUG
                printf("    tar_append_file(): encoding symlink \"%s\" -> "
                       "\"%s\"...\n", realname, path);
 #endif
                th_set_link(t, path);
+               free(path);
        }
 
        /* print file info */
--- a/lib/decode.c
+++ b/lib/decode.c
@@ -29,10 +29,13 @@ th_get_pathname(TAR *t)
        if (t->th_buf.gnu_longname)
                return t->th_buf.gnu_longname;
 
+       size_t pathlen =
+         strlen(t->th_buf.prefix) + strlen(t->th_buf.name) + 2;
+
        /* allocate the th_pathname buffer if not already */
        if (t->th_pathname == NULL)
        {
-               t->th_pathname = malloc(MAXPATHLEN * sizeof(char));
+               t->th_pathname = malloc(pathlen);
                if (t->th_pathname == NULL)
                        /* out of memory */
                        return NULL;
@@ -40,11 +43,11 @@ th_get_pathname(TAR *t)
 
        if (t->th_buf.prefix[0] == '\0')
        {
-               snprintf(t->th_pathname, MAXPATHLEN, "%.100s", t->th_buf.name);
+               snprintf(t->th_pathname, pathlen, "%.100s", t->th_buf.name);
        }
        else
        {
-               snprintf(t->th_pathname, MAXPATHLEN, "%.155s/%.100s",
+               snprintf(t->th_pathname, pathlen, "%.155s/%.100s",
                         t->th_buf.prefix, t->th_buf.name);
        }
 
--- a/lib/util.c
+++ b/lib/util.c
@@ -15,6 +15,7 @@
 #include <stdio.h>
 #include <sys/param.h>
 #include <errno.h>
+#include <stdlib.h>
 
 #ifdef STDC_HEADERS
 # include <string.h>
@@ -25,13 +26,15 @@
 int
 path_hashfunc(char *key, int numbuckets)
 {
-       char buf[MAXPATHLEN];
+       char *buf;
        char *p;
+       int i;
 
-       strcpy(buf, key);
+       buf = strdup(key);
        p = basename(buf);
-
-       return (((unsigned int)p[0]) % numbuckets);
+       i = ((unsigned int)p[0]) % numbuckets;
+       free(buf);
+       return (i);
 }
 
 
@@ -77,15 +80,26 @@ ino_hash(ino_t *inode)
 int
 mkdirhier(char *path)
 {
-       char src[MAXPATHLEN], dst[MAXPATHLEN] = "";
-       char *dirp, *nextp = src;
-       int retval = 1;
+       char *src, *dst = NULL;
+       char *dirp, *nextp = NULL;
+       int retval = 1, len;
+
+       len = strlen(path);
+       if ((src = strdup(path)) == NULL)
+       {
+               errno = ENOMEM;
+               return -1;
+       }
+       nextp = src;
 
-       if (strlcpy(src, path, sizeof(src)) > sizeof(src))
+       /* Make room for // with absolute paths */
+       if ((dst = malloc(len + 2)) == NULL)
        {
-               errno = ENAMETOOLONG;
+               free(src);
+               errno = ENOMEM;
                return -1;
        }
+       dst[0] = '\0';
 
        if (path[0] == '/')
                strcpy(dst, "/");
@@ -102,12 +116,18 @@ mkdirhier(char *path)
                if (mkdir(dst, 0777) == -1)
                {
                        if (errno != EEXIST)
+                       {
+                               free(src);
+                               free(dst);
                                return -1;
+                       }
                }
                else
                        retval = 0;
        }
 
+       free(src);
+       free(dst);
        return retval;
 }
 
--- a/lib/wrapper.c
+++ b/lib/wrapper.c
@@ -16,18 +16,18 @@
 #include <sys/param.h>
 #include <dirent.h>
 #include <errno.h>
+#include <stdlib.h>
 
 #ifdef STDC_HEADERS
 # include <string.h>
 #endif
 
-
 int
 tar_extract_glob(TAR *t, char *globname, char *prefix)
 {
        char *filename;
-       char buf[MAXPATHLEN];
-       int i;
+       char *buf = NULL;
+       int i, len;
 
        while ((i = th_read(t)) == 0)
        {
@@ -41,11 +41,25 @@ tar_extract_glob(TAR *t, char *globname,
                if (t->options & TAR_VERBOSE)
                        th_print_long_ls(t);
                if (prefix != NULL)
-                       snprintf(buf, sizeof(buf), "%s/%s", prefix, filename);
+               {
+                       len = strlen(prefix) + 1 + strlen(filename);
+                       if ((buf = malloc(len + 1)) == NULL)
+                               return -1;
+                       sprintf(buf, "%s/%s", prefix, filename);
+               }
                else
-                       strlcpy(buf, filename, sizeof(buf));
+               {
+                       len = strlen(filename);
+                       if ((buf = malloc(len + 1)) == NULL)
+                               return -1;
+                       strcpy(buf, filename);
+               }
                if (tar_extract_file(t, buf) != 0)
+               {
+                       free(buf);
                        return -1;
+               }
+               free(buf);
        }
 
        return (i == 1 ? 0 : -1);
@@ -56,8 +70,9 @@ int
 tar_extract_all(TAR *t, char *prefix)
 {
        char *filename;
-       char buf[MAXPATHLEN];
-       int i;
+       char *buf = NULL;
+       size_t bufsize = 0;
+       int i, len;
 
 #ifdef DEBUG
        printf("==> tar_extract_all(TAR *t, \"%s\")\n",
@@ -69,19 +84,34 @@ tar_extract_all(TAR *t, char *prefix)
 #ifdef DEBUG
                puts("    tar_extract_all(): calling th_get_pathname()");
 #endif
+
                filename = th_get_pathname(t);
                if (t->options & TAR_VERBOSE)
                        th_print_long_ls(t);
                if (prefix != NULL)
-                       snprintf(buf, sizeof(buf), "%s/%s", prefix, filename);
+               {
+                       len = strlen(prefix) + 1 + strlen(filename);
+                       if ((buf = malloc(len + 1)) == NULL)
+                               return -1;
+                       sprintf(buf, "%s/%s", prefix, filename);
+               }
                else
-                       strlcpy(buf, filename, sizeof(buf));
+               {
+                       len = strlen(filename);
+                       if ((buf = malloc(len + 1)) == NULL)
+                               return -1;
+                       strcpy(buf, filename);
+               }
 #ifdef DEBUG
                printf("    tar_extract_all(): calling tar_extract_file(t, "
                       "\"%s\")\n", buf);
 #endif
                if (tar_extract_file(t, buf) != 0)
+               {
+                       free(buf);
                        return -1;
+               }
+               free(buf);
        }
 
        return (i == 1 ? 0 : -1);
@@ -91,11 +121,14 @@ tar_extract_all(TAR *t, char *prefix)
 int
 tar_append_tree(TAR *t, char *realdir, char *savedir)
 {
-       char realpath[MAXPATHLEN];
-       char savepath[MAXPATHLEN];
+       char *realpath = NULL;
+       size_t realpathsize = 0;
+       char *savepath = NULL;
+       size_t savepathsize = 0;
        struct dirent *dent;
        DIR *dp;
        struct stat s;
+       int len;
 
 #ifdef DEBUG
        printf("==> tar_append_tree(0x%lx, \"%s\", \"%s\")\n",
@@ -122,11 +155,19 @@ tar_append_tree(TAR *t, char *realdir, c
                    strcmp(dent->d_name, "..") == 0)
                        continue;
 
-               snprintf(realpath, MAXPATHLEN, "%s/%s", realdir,
+               len = strlen(realdir) + 1 + strlen(dent->d_name);
+               if ((realpath = malloc(len + 1)) == NULL)
+                       return -1;
+               snprintf(realpath, len + 1, "%s/%s", realdir,
                         dent->d_name);
                if (savedir)
-                       snprintf(savepath, MAXPATHLEN, "%s/%s", savedir,
+               {
+                       len = strlen(savedir) + 1 + strlen(dent->d_name);
+                       if ((savepath = malloc(len + 1)) == NULL)
+                               return -1;
+                       snprintf(realpath, len + 1, "%s/%s", savedir,
                                 dent->d_name);
+               }
 
                if (lstat(realpath, &s) != 0)
                        return -1;
@@ -135,13 +176,23 @@ tar_append_tree(TAR *t, char *realdir, c
                {
                        if (tar_append_tree(t, realpath,
                                            (savedir ? savepath : NULL)) != 0)
+                       {
+                               free(realpath);
+                               free(savepath);
                                return -1;
+                       }
                        continue;
                }
 
                if (tar_append_file(t, realpath,
                                    (savedir ? savepath : NULL)) != 0)
+               {
+                       free(realpath);
+                       free(savepath);
                        return -1;
+               }
+               free(realpath);
+               free(savepath);
        }
 
        closedir(dp);
--- a/libtar/libtar.c
+++ b/libtar/libtar.c
@@ -111,8 +111,9 @@ create(char *tarfile, char *rootdir, lib
 {
        TAR *t;
        char *pathname;
-       char buf[MAXPATHLEN];
+       char *buf = NULL;
        libtar_listptr_t lp;
+       int len;
 
        if (tar_open(&t, tarfile,
 #ifdef HAVE_LIBZ
@@ -133,17 +134,29 @@ create(char *tarfile, char *rootdir, lib
        {
                pathname = (char *)libtar_listptr_data(&lp);
                if (pathname[0] != '/' && rootdir != NULL)
-                       snprintf(buf, sizeof(buf), "%s/%s", rootdir, pathname);
+               {
+                       len = strlen(rootdir) + 1 + strlen(pathname);
+                       if ((buf = malloc(len + 1)) == NULL)
+                               return -1;
+                       snprintf(buf, len + 1, "%s/%s", rootdir, pathname);
+               }
                else
-                       strlcpy(buf, pathname, sizeof(buf));
+               {
+                       len = strlen(pathname);
+                       if ((buf = malloc(len + 1)) == NULL)
+                               return -1;
+                       strlcpy(buf, pathname, len + 1);
+               }
                if (tar_append_tree(t, buf, pathname) != 0)
                {
                        fprintf(stderr,
                                "tar_append_tree(\"%s\", \"%s\"): %s\n", buf,
                                pathname, strerror(errno));
                        tar_close(t);
+                       free(buf);
                        return -1;
                }
+               free(buf);
        }
 
        if (tar_append_eof(t) != 0)