diff options
| author | George Burgess IV <gbiv@google.com> | 2021-02-19 13:01:33 -0800 |
|---|---|---|
| committer | George Burgess IV <gbiv@google.com> | 2021-02-19 13:04:07 -0800 |
| commit | a263a88381a3fad800dbdfc8792c89b4c94b8a21 (patch) | |
| tree | d956e3f2eb92372181ad5398030ee8e91fe78cab /trusty/coverage/coverage_test.cpp | |
| parent | d57652f15b078d06d9258094f8a951374a677f2c (diff) | |
snapuserd: fix a use of an uninitialized value
`header` is only initialized in the `if` block of this condition. Hence,
its use in the `else` portion isn't correct. Refactor the code a bit to
make this kind of bug a bit harder to write in the future.
Caught by the static analyzer:
> system/core/fs_mgr/libsnapshot/snapuserd.cpp:457:9: warning: 1st
function call argument is an uninitialized value
[clang-analyzer-core.CallAndMessage]
Bug: None
Test: TreeHugger
Change-Id: Ie56578520acf3cc972efa3336e40698feed20200
Diffstat (limited to 'trusty/coverage/coverage_test.cpp')
0 files changed, 0 insertions, 0 deletions