Merge "app_main: silence static analyzer warnings" am: cf188f664a
am: 8a4be2a6ae
Change-Id: Ic7b6e1092261a0e76fc84b213166729ea3451166
This commit is contained in:
@@ -235,6 +235,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;
|
||||
@@ -258,6 +261,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]);
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user