|
 |
de1ab5 |
autofs-5.1.8 - fix minus only option handling in concat_options()
|
|
|
de1ab5 |
|
|
|
de1ab5 |
From: Ian Kent <raven@themaw.net>
|
|
|
de1ab5 |
|
|
|
de1ab5 |
While a '-' alone isn't strictly valid it hadn't previously cuased a
|
|
|
de1ab5 |
parse error. So commit 9047e91ffa69 (autofs-5.1.7 - fix concat_options()
|
|
|
de1ab5 |
error handling) introduced a regression by no longer allowing this.
|
|
|
de1ab5 |
|
|
|
de1ab5 |
Fix this regression by only failing if errno is set to a non-zero value
|
|
|
de1ab5 |
on return from concat_options() as well as returning NULL.
|
|
|
de1ab5 |
|
|
|
de1ab5 |
Fixes: 9047e91ffa69 (autofs-5.1.7 - fix concat_options() error handling)
|
|
|
de1ab5 |
Signed-off-by: Ian Kent <raven@themaw.net>
|
|
|
de1ab5 |
---
|
|
|
de1ab5 |
CHANGELOG | 1 +
|
|
|
de1ab5 |
modules/parse_sun.c | 25 +++++++++++++++++++------
|
|
|
de1ab5 |
2 files changed, 20 insertions(+), 6 deletions(-)
|
|
|
de1ab5 |
|
|
|
de1ab5 |
--- autofs-5.1.7.orig/CHANGELOG
|
|
|
de1ab5 |
+++ autofs-5.1.7/CHANGELOG
|
|
|
de1ab5 |
@@ -110,6 +110,7 @@
|
|
|
de1ab5 |
- fix hosts map deadlock on restart.
|
|
|
de1ab5 |
- fix deadlock with hosts map reload.
|
|
|
de1ab5 |
- fix memory leak in update_hosts_mounts().
|
|
|
de1ab5 |
+- fix minus only option handling in concat_options().
|
|
|
de1ab5 |
|
|
|
de1ab5 |
25/01/2021 autofs-5.1.7
|
|
|
de1ab5 |
- make bind mounts propagation slave by default.
|
|
|
de1ab5 |
--- autofs-5.1.7.orig/modules/parse_sun.c
|
|
|
de1ab5 |
+++ autofs-5.1.7/modules/parse_sun.c
|
|
|
de1ab5 |
@@ -376,10 +376,16 @@ static int do_init(int argc, const char
|
|
|
de1ab5 |
if (gbl_options) {
|
|
|
de1ab5 |
append_options = defaults_get_append_options();
|
|
|
de1ab5 |
if (append_options) {
|
|
|
de1ab5 |
- char *tmp = concat_options(gbl_options, ctxt->optstr);
|
|
|
de1ab5 |
+ char *tmp;
|
|
|
de1ab5 |
+
|
|
|
de1ab5 |
+ errno = 0;
|
|
|
de1ab5 |
+ tmp = concat_options(gbl_options, ctxt->optstr);
|
|
|
de1ab5 |
if (!tmp) {
|
|
|
de1ab5 |
- char *estr = strerror_r(errno, buf, MAX_ERR_BUF);
|
|
|
de1ab5 |
- logerr(MODPREFIX "concat_options: %s", estr);
|
|
|
de1ab5 |
+ /* Ignore non-error NULL return */
|
|
|
de1ab5 |
+ if (errno) {
|
|
|
de1ab5 |
+ char *estr = strerror_r(errno, buf, MAX_ERR_BUF);
|
|
|
de1ab5 |
+ logerr(MODPREFIX "concat_options: %s", estr);
|
|
|
de1ab5 |
+ }
|
|
|
de1ab5 |
/* freed in concat_options */
|
|
|
de1ab5 |
ctxt->optstr = NULL;
|
|
|
de1ab5 |
} else
|
|
|
de1ab5 |
@@ -1007,9 +1013,12 @@ static int parse_mapent(const char *ent,
|
|
|
de1ab5 |
free(myoptions);
|
|
|
de1ab5 |
myoptions = newopt;
|
|
|
de1ab5 |
} else if (newopt) {
|
|
|
de1ab5 |
+ errno = 0;
|
|
|
de1ab5 |
tmp = concat_options(myoptions, newopt);
|
|
|
de1ab5 |
- if (!tmp) {
|
|
|
de1ab5 |
+ /* Ignore non-error NULL return */
|
|
|
de1ab5 |
+ if (!tmp && errno) {
|
|
|
de1ab5 |
char *estr;
|
|
|
de1ab5 |
+
|
|
|
de1ab5 |
estr = strerror_r(errno, buf, MAX_ERR_BUF);
|
|
|
de1ab5 |
error(logopt, MODPREFIX
|
|
|
de1ab5 |
"concat_options: %s", estr);
|
|
|
de1ab5 |
@@ -1381,8 +1390,10 @@ dont_expand:
|
|
|
de1ab5 |
free(mnt_options);
|
|
|
de1ab5 |
mnt_options = noptions;
|
|
|
de1ab5 |
} else if (noptions) {
|
|
|
de1ab5 |
+ errno = 0;
|
|
|
de1ab5 |
tmp = concat_options(mnt_options, noptions);
|
|
|
de1ab5 |
- if (!tmp) {
|
|
|
de1ab5 |
+ /* Ignore non-error NULL return */
|
|
|
de1ab5 |
+ if (!tmp && errno) {
|
|
|
de1ab5 |
char *estr = strerror_r(errno, buf, MAX_ERR_BUF);
|
|
|
de1ab5 |
error(ap->logopt,
|
|
|
de1ab5 |
MODPREFIX "concat_options: %s", estr);
|
|
|
de1ab5 |
@@ -1406,8 +1417,10 @@ dont_expand:
|
|
|
de1ab5 |
free(options);
|
|
|
de1ab5 |
options = mnt_options;
|
|
|
de1ab5 |
} else if (mnt_options) {
|
|
|
de1ab5 |
+ errno = 0;
|
|
|
de1ab5 |
tmp = concat_options(options, mnt_options);
|
|
|
de1ab5 |
- if (!tmp) {
|
|
|
de1ab5 |
+ /* Ignore non-error NULL return */
|
|
|
de1ab5 |
+ if (!tmp && errno) {
|
|
|
de1ab5 |
char *estr = strerror_r(errno, buf, MAX_ERR_BUF);
|
|
|
de1ab5 |
error(ap->logopt, MODPREFIX "concat_options: %s", estr);
|
|
|
de1ab5 |
free(pmapent);
|