Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "app_main: silence static analyzer warnings"

Browse Source
This commit is contained in:
Treehugger Robot
2017-07-12 02:43:14 +00:00
committed by Gerrit Code Review
parent 3eff26449f c29844d072
commit cf188f664a
1 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
cmds/app_process/app_main.cpp
View File

@@ -233,6 +233,9 @@ int main(int argc, char* const argv[])
for (i = 0; i < argc; i++) { for (i = 0; i < argc; i++) {
if (known_command == true) { if (known_command == true) {
runtime.addOption(strdup(argv[i])); 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]); ALOGV("app_process main add known option '%s'", argv[i]);
known_command = false; known_command = false;
continue; continue;
@@ -256,6 +259,9 @@ int main(int argc, char* const argv[])
} }
runtime.addOption(strdup(argv[i])); 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]); ALOGV("app_process main add option '%s'", argv[i]);
} }