Commit 9a5ad138 authored by Benjamin Marzinski's avatar Benjamin Marzinski Committed by Steven Whitehouse

[GFS2] Add NULL entry to token table

match_token() was returning garbage data instead of a fail value. This data
happened to match a valid option id for an option that required an argument (in
this case, lockproto=%s) For match_token() to correctly fail if the option
doesn't match any of the tokens, the token table must end with a NULL entry.
This patch adds the NULL entry.
Signed-off-by: default avatarBenjamin E. Marzinski <bmarzins@redhat.com>
Signed-off-by: default avatarSteven Whitehouse <swhiteho@redhat.com>
parent 382e6e25
...@@ -42,6 +42,7 @@ enum { ...@@ -42,6 +42,7 @@ enum {
Opt_nosuiddir, Opt_nosuiddir,
Opt_data_writeback, Opt_data_writeback,
Opt_data_ordered, Opt_data_ordered,
Opt_err,
}; };
static match_table_t tokens = { static match_table_t tokens = {
...@@ -64,7 +65,8 @@ static match_table_t tokens = { ...@@ -64,7 +65,8 @@ static match_table_t tokens = {
{Opt_suiddir, "suiddir"}, {Opt_suiddir, "suiddir"},
{Opt_nosuiddir, "nosuiddir"}, {Opt_nosuiddir, "nosuiddir"},
{Opt_data_writeback, "data=writeback"}, {Opt_data_writeback, "data=writeback"},
{Opt_data_ordered, "data=ordered"} {Opt_data_ordered, "data=ordered"},
{Opt_err, NULL}
}; };
/** /**
...@@ -237,6 +239,7 @@ int gfs2_mount_args(struct gfs2_sbd *sdp, char *data_arg, int remount) ...@@ -237,6 +239,7 @@ int gfs2_mount_args(struct gfs2_sbd *sdp, char *data_arg, int remount)
case Opt_data_ordered: case Opt_data_ordered:
args->ar_data = GFS2_DATA_ORDERED; args->ar_data = GFS2_DATA_ORDERED;
break; break;
case Opt_err:
default: default:
fs_info(sdp, "unknown option: %s\n", o); fs_info(sdp, "unknown option: %s\n", o);
error = -EINVAL; error = -EINVAL;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment