Fixes a bug in option string parsing wherein a line such as: ro,relatime,,nosuid,nodev Will be seen as only the tokens "ro" and "relatime" after the parser encounters a zero length (and erroneously declared NULL) option. Signed-off-by: Dave Reisner <dreisner@xxxxxxxxxxxxx> --- libmount/src/optstr.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/libmount/src/optstr.c b/libmount/src/optstr.c index af5953b..66d2a06 100644 --- a/libmount/src/optstr.c +++ b/libmount/src/optstr.c @@ -62,6 +62,11 @@ static int mnt_optstr_parse_next(char **optstr, char **name, size_t *namesz, if (valsz) *valsz = 0; + /* trim leading commas as to not invalidate option + * strings with multiple consecutive commas */ + while (optstr0 && *optstr0 == ',') + optstr0++; + for (p = optstr0; p && *p; p++) { if (!start) start = p; /* begin of the option item */ -- 1.7.10.3 -- To unsubscribe from this list: send the line "unsubscribe util-linux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html