summaryrefslogtreecommitdiffstats
path: root/usr.sbin/jls
diff options
context:
space:
mode:
authorjamie <jamie@FreeBSD.org>2015-02-22 00:00:10 +0000
committerjamie <jamie@FreeBSD.org>2015-02-22 00:00:10 +0000
commit2aa717eb37d45980269447d27ee3695f6e4ee3b1 (patch)
tree4e490083bd289c7d37fdd38dad91a9fa509833b6 /usr.sbin/jls
parent0ea1b83e3703e39c50804c558e396324cec66c74 (diff)
downloadFreeBSD-src-2aa717eb37d45980269447d27ee3695f6e4ee3b1.zip
FreeBSD-src-2aa717eb37d45980269447d27ee3695f6e4ee3b1.tar.gz
Allow for parameters added with the JP_OPT flag to not exist.
That's why the flag exists in the first place. MFC after: 1 week
Diffstat (limited to 'usr.sbin/jls')
-rw-r--r--usr.sbin/jls/jls.c7
1 files changed, 3 insertions, 4 deletions
diff --git a/usr.sbin/jls/jls.c b/usr.sbin/jls/jls.c
index 8e5512d..b1e28fb 100644
--- a/usr.sbin/jls/jls.c
+++ b/usr.sbin/jls/jls.c
@@ -294,10 +294,8 @@ add_param(const char *name, void *value, size_t valuelen,
param->jp_flags |= flags;
return param - params;
}
- if (jailparam_init(param, name) < 0)
- errx(1, "%s", jail_errmsg);
- param->jp_flags = flags;
- if ((value != NULL ? jailparam_import_raw(param, value, valuelen)
+ if (jailparam_init(param, name) < 0 ||
+ (value != NULL ? jailparam_import_raw(param, value, valuelen)
: jailparam_import(param, value)) < 0) {
if (flags & JP_OPT) {
nparams--;
@@ -305,6 +303,7 @@ add_param(const char *name, void *value, size_t valuelen,
}
errx(1, "%s", jail_errmsg);
}
+ param->jp_flags = flags;
return param - params;
}
OpenPOWER on IntegriCloud