diff options
author | trasz <trasz@FreeBSD.org> | 2015-03-12 12:17:15 +0000 |
---|---|---|
committer | trasz <trasz@FreeBSD.org> | 2015-03-12 12:17:15 +0000 |
commit | e47fa13bc5cc0ec18a4fb98088cb107d470fb600 (patch) | |
tree | 818ff5360d14192ee02273df37566233dd759287 /usr.sbin/autofs | |
parent | 724f53c3cf42294fa961bd4c6033b9360270395b (diff) | |
download | FreeBSD-src-e47fa13bc5cc0ec18a4fb98088cb107d470fb600.zip FreeBSD-src-e47fa13bc5cc0ec18a4fb98088cb107d470fb600.tar.gz |
Make concat() accept NULL arguments.
MFC after: 1 month
Sponsored by: The FreeBSD Foundation
Diffstat (limited to 'usr.sbin/autofs')
-rw-r--r-- | usr.sbin/autofs/automount.c | 10 | ||||
-rw-r--r-- | usr.sbin/autofs/automountd.c | 9 | ||||
-rw-r--r-- | usr.sbin/autofs/common.c | 6 |
3 files changed, 8 insertions, 17 deletions
diff --git a/usr.sbin/autofs/automount.c b/usr.sbin/autofs/automount.c index 8a7c91a..8bcc79b 100644 --- a/usr.sbin/autofs/automount.c +++ b/usr.sbin/autofs/automount.c @@ -345,11 +345,7 @@ main_automount(int argc, char **argv) force_unmount = true; break; case 'o': - if (options == NULL) { - options = checked_strdup(optarg); - } else { - options = concat(options, ',', optarg); - } + options = concat(options, ',', optarg); break; case 'u': do_unmount = true; @@ -385,9 +381,7 @@ main_automount(int argc, char **argv) parse_master(root, AUTO_MASTER_PATH); if (show_maps) { - if (options != NULL) { - root->n_options = concat(options, ',', root->n_options); - } + root->n_options = concat(options, ',', root->n_options); if (show_maps > 1) { node_expand_indirect_maps(root); node_expand_ampersand(root, NULL); diff --git a/usr.sbin/autofs/automountd.c b/usr.sbin/autofs/automountd.c index 191e712..9c399fd 100644 --- a/usr.sbin/autofs/automountd.c +++ b/usr.sbin/autofs/automountd.c @@ -241,8 +241,7 @@ handle_request(const struct autofs_daemon_request *adr, char *cmdline_options, /* * Prepend options passed via automountd(8) command line. */ - if (cmdline_options != NULL) - options = concat(cmdline_options, ',', options); + options = concat(cmdline_options, ',', options); if (node->n_location == NULL) { log_debugx("found node defined at %s:%d; not a mountpoint", @@ -455,11 +454,7 @@ main_automountd(int argc, char **argv) maxproc = atoi(optarg); break; case 'o': - if (options == NULL) { - options = checked_strdup(optarg); - } else { - options = concat(options, ',', optarg); - } + options = concat(options, ',', optarg); break; case 'v': debug++; diff --git a/usr.sbin/autofs/common.c b/usr.sbin/autofs/common.c index d725eab..7c744d3 100644 --- a/usr.sbin/autofs/common.c +++ b/usr.sbin/autofs/common.c @@ -94,8 +94,10 @@ concat(const char *s1, char separator, const char *s2) char *result; int ret; - assert(s1 != NULL); - assert(s2 != NULL); + if (s1 == NULL) + s1 = ""; + if (s2 == NULL) + s2 = ""; /* * If s2 starts with separator - skip it; otherwise concatenating |