In sdbm_open(), replace strcat()s and strcpy()s with memcpy()s.
authorNicholas Clark <nick@ccl4.org>
Fri, 3 Apr 2009 19:41:48 +0000 (20:41 +0100)
committerDavid Mitchell <davem@iabyn.com>
Mon, 20 Apr 2009 00:06:57 +0000 (01:06 +0100)
(cherry picked from commit 081f72ad6fa2b76e0b3cd9046371b2dbd9130114)

ext/SDBM_File/sdbm/sdbm.c

index 8be9bb9..c5f7aa8 100644 (file)
@@ -77,23 +77,28 @@ sdbm_open(register char *file, register int flags, register int mode)
        register DBM *db;
        register char *dirname;
        register char *pagname;
-       register int n;
+       size_t filelen;
+       const size_t dirfext_len = sizeof(DIRFEXT "");
+       const size_t pagfext_len = sizeof(PAGFEXT "");
 
        if (file == NULL || !*file)
                return errno = EINVAL, (DBM *) NULL;
 /*
  * need space for two seperate filenames
  */
-       n = strlen(file) * 2 + strlen(DIRFEXT) + strlen(PAGFEXT) + 2;
+       filelen = strlen(file);
 
-       if ((dirname = (char *) malloc((unsigned) n)) == NULL)
+       if ((dirname = (char *) malloc(filelen + dirfext_len + 1
+                                      + filelen + pagfext_len + 1)) == NULL)
                return errno = ENOMEM, (DBM *) NULL;
 /*
  * build the file names
  */
-       strcat(strcpy(dirname, file), DIRFEXT);
-       pagname = strcpy(dirname + strlen(dirname) + 1, file);
-       strcat(pagname, PAGFEXT);
+       memcpy(dirname, file, filelen);
+       memcpy(dirname + filelen, DIRFEXT, dirfext_len + 1);
+       pagname = dirname + filelen + dirfext_len + 1;
+       memcpy(pagname, file, filelen);
+       memcpy(pagname + filelen, PAGFEXT, pagfext_len + 1);
 
        db = sdbm_prep(dirname, pagname, flags, mode);
        free((char *) dirname);