On Wed, Aug 21, 2019 at 09:22:49PM -0700, Eric Biggers wrote: > > > diff --git a/fs/fs_parser.c b/fs/fs_parser.c > > > index 83b66c9e9a24..7498a44f18c0 100644 > > > --- a/fs/fs_parser.c > > > +++ b/fs/fs_parser.c > > > @@ -206,6 +206,9 @@ int fs_parse(struct fs_context *fc, > > > case fs_param_is_fd: { > > > switch (param->type) { > > > case fs_value_is_string: > > > + if (!result->has_value) > > > + goto bad_value; > > > + > > > ret = kstrtouint(param->string, 0, &result->uint_32); > > > break; > > > case fs_value_is_file: > > > -- > > > 2.17.1 > > > > Reviewed-by: Eric Biggers <ebiggers@xxxxxxxxxx> > > > > Al, can you please apply this patch? > > > > - Eric > > Ping. Al, when are you going to apply this? Sits in the local queue. Sorry, got seriously sidetracked into configfs mess lately, will update for-next tomorrow and push it out.