diff options
author | Mike Frysinger <vapier@google.com> | 2018-02-06 18:31:57 +0000 |
---|---|---|
committer | android-build-merger <android-build-merger@google.com> | 2018-02-06 18:31:57 +0000 |
commit | 8e193ad5123f0929ddb865a49a665f79724f2fa9 (patch) | |
tree | e12ffa400796237bd2e5c0c5d04005ac15733875 | |
parent | 7640664b6dfcada0b24fd0912acf7d2fbfb784c0 (diff) | |
parent | 450c08b69210d7a2093432e1b5b5d91e3a7dc28a (diff) | |
download | minijail-8e193ad5123f0929ddb865a49a665f79724f2fa9.tar.gz |
minijail0_cli_unittest: fix random crashes/failures am: 9741372f23 am: 0d40743b67
am: 450c08b692
Change-Id: I8a6e71d6a703bddaccc36887c8a89fcd7da6d30b
-rw-r--r-- | minijail0_cli_unittest.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/minijail0_cli_unittest.cc b/minijail0_cli_unittest.cc index a774d55..2427a01 100644 --- a/minijail0_cli_unittest.cc +++ b/minijail0_cli_unittest.cc @@ -42,8 +42,10 @@ class CliTest : public ::testing::Test { // as it parses things (which is normally permissible with argv). int parse_args_(const std::vector<std::string>& argv, int *exit_immediately, ElfType *elftype) { - // Make sure we reset the getopts state when scanning a new argv. - optind = 1; + // Make sure we reset the getopts state when scanning a new argv. Setting + // this to 0 is a GNU extension, but AOSP/BSD also checks this (as an alias + // to their "optreset"). + optind = 0; std::vector<const char *> pargv; pargv.push_back("minijail0"); |