Author: ed
Date: Tue May 31 06:45:19 2016
New Revision: 301024
URL: https://svnweb.freebsd.org/changeset/base/301024

Log:
  Stop using the non-standard basename_r() function.
  
  This change makes the code use the POSIX basename() function. It has the
  advantage that (if implemented correctly), it also imposes no restrict
  on the pathname length.
  
  Notice that I haven't added any error handling to the strdup() call. It
  looks like none of the other calls to strdup() and malloc() performed by
  this utility do it either.
  
  Reviewed by:  hrs
  Differential Revision:        https://reviews.freebsd.org/D6626

Modified:
  head/sbin/swapon/swapon.c

Modified: head/sbin/swapon/swapon.c
==============================================================================
--- head/sbin/swapon/swapon.c   Tue May 31 06:24:09 2016        (r301023)
+++ head/sbin/swapon/swapon.c   Tue May 31 06:45:19 2016        (r301024)
@@ -216,7 +216,7 @@ main(int argc, char **argv)
 static const char *
 swap_on_off(const char *name, int doingall, char *mntops)
 {
-       char base[PATH_MAX];
+       char *base, *basebuf;
 
        /* Swap on vnode-backed md(4) device. */
        if (mntops != NULL &&
@@ -227,17 +227,23 @@ swap_on_off(const char *name, int doinga
             strncmp(MD_NAME, name, sizeof(MD_NAME)) == 0))
                return (swap_on_off_md(name, mntops, doingall));
 
-       basename_r(name, base);
+       basebuf = strdup(name);
+       base = basename(basebuf);
 
        /* Swap on encrypted device by GEOM_BDE. */
-       if (fnmatch("*.bde", base, 0) == 0)
+       if (fnmatch("*.bde", base, 0) == 0) {
+               free(basebuf);
                return (swap_on_off_gbde(name, doingall));
+       }
 
        /* Swap on encrypted device by GEOM_ELI. */
-       if (fnmatch("*.eli", base, 0) == 0)
+       if (fnmatch("*.eli", base, 0) == 0) {
+               free(basebuf);
                return (swap_on_off_geli(name, mntops, doingall));
+       }
 
        /* Swap on special file. */
+       free(basebuf);
        return (swap_on_off_sfile(name, doingall));
 }
 
_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to