diff options
| author | Xin Li <delphij@google.com> | 2019-01-09 15:15:15 -0800 |
|---|---|---|
| committer | Xin Li <delphij@google.com> | 2019-01-09 23:26:13 +0000 |
| commit | 3a039ff76350811822b3ab885c64c683016da879 (patch) | |
| tree | 22ebc7992857e175b3bfd6f29704a2bb327b6f53 /tests/DynamicCodeLoggerIntegrationTests/src/cpp/test_executable.cpp | |
| parent | dcfef16295126c0197e5e1f6449280efdf787fea (diff) | |
Fix compile time warning.
In checkIncidentPermissions(), the intention of code is to bail out when
the caller do not have appropriate permission.
Logically, DEST_LOCAL and DEST_EXPLICIT should be checked separately.
In the current code, we would fall-through to DEST_EXPLICIT case, but
because permitted DEST_LOCAL callers (AID_SHELL and AID_ROOT) are also
permitted DEST_EXPLICIT callers, we will still get the Status:ok().
Test: build
Change-Id: I0f6121cb1aba7724306b99469f5ef97b4358ac7d
Diffstat (limited to 'tests/DynamicCodeLoggerIntegrationTests/src/cpp/test_executable.cpp')
0 files changed, 0 insertions, 0 deletions