tree 4a3ddf36ac311cff4c5e65bcdf20f3a2ad86006c
parent f3bd69ae0fd51f8ec16b25293992bfbbf83cb3ea
author Mike Frysinger <vapier@google.com> 1516226208 -0500
committer Treehugger Robot <treehugger-gerrit@google.com> 1516315109 +0000

fix handling of consecutive optional arguments

The C library strtok function will skip blank tokens.  e.g. "a;;b"
will yield ["a", "b"], not ["a", "", "b"].  This affects some of our
command line options (like -k and -b), so switch them to our own
tokenize function which will yield empty strings as needed.

This also lets us make sure the user doesn't pass too many arguments.
Currently we were accepting (and ignoring the extra parts of) options
like -b /var,/var,1,foo,bar,fat,cat.  This would be treated like we
passed in -b /var,/var,1.

Bug: None
Test: `minijail0 --profile minimalistic-mountns -b /dev/log,,1 /bin/ls /dev/log` works

Change-Id: I653deb59ac8e6b1df108e1062a87dd87e7cb8a17