dm verity: fix verity_parse_opt_args parsing
authorMike Snitzer <snitzer@kernel.org>
Tue, 9 Aug 2022 22:07:28 +0000 (18:07 -0400)
committerMike Snitzer <snitzer@kernel.org>
Tue, 9 Aug 2022 23:15:17 +0000 (19:15 -0400)
commitf876df9f12cda68e68995b33b36491d78fd3ecce
tree532890b9182551c4a7869767e1ddfc4477ad4fe5
parent8c22816dbc8733ea761c3b74ec62d9463e242c56
dm verity: fix verity_parse_opt_args parsing

Commit df326e7a0699 ("dm verity: allow optional args to alter primary
args handling") introduced a bug where verity_parse_opt_args() wouldn't
properly shift past an optional argument's additional params (by
ignoring them).

Fix this by avoiding returning with error if an unknown argument is
encountered when @only_modifier_opts=true is passed to
verity_parse_opt_args().

In practice this regressed the cryptsetup testsuite's FEC testing
because unknown optional arguments were encountered, wherey
short-circuiting ever testing FEC mode. With this fix all of the
cryptsetup testsuite's verity FEC tests pass.

Fixes: df326e7a0699 ("dm verity: allow optional args to alter primary args handling")
Reported-by: Milan Broz <gmazyland@gmail.com>>
Signed-off-by: Mike Snitzer <snitzer@kernel.org>
drivers/md/dm-verity-target.c