Merge "app_main: silence static analyzer warnings"

This commit is contained in:
Treehugger Robot
2017-07-12 02:43:14 +00:00
committed by Gerrit Code Review

View File

@@ -233,6 +233,9 @@ int main(int argc, char* const argv[])
for (i = 0; i < argc; i++) {
if (known_command == true) {
runtime.addOption(strdup(argv[i]));
// The static analyzer gets upset that we don't ever free the above
// string. Since the allocation is from main, leaking it doesn't seem
// problematic. NOLINTNEXTLINE
ALOGV("app_process main add known option '%s'", argv[i]);
known_command = false;
continue;
@@ -256,6 +259,9 @@ int main(int argc, char* const argv[])
}
runtime.addOption(strdup(argv[i]));
// The static analyzer gets upset that we don't ever free the above
// string. Since the allocation is from main, leaking it doesn't seem
// problematic. NOLINTNEXTLINE
ALOGV("app_process main add option '%s'", argv[i]);
}