diff options
author | 2015-02-26 13:28:53 -0800 | |
---|---|---|
committer | 2015-02-26 13:28:53 -0800 | |
commit | 71d7f7f002332b9904bf12eb3f1908883f99120e (patch) | |
tree | d52a7b9bd64a97be9d006aa016ffb8f2cf928a2a /src/utils/mac/SkCreateCGImageRef.cpp | |
parent | e6cf9cb68511ff08156f834859db39232eb37be8 (diff) |
Initialize three locals to avoid the risk of uninitialized reads.
/analyze recently reported two new warnings about reading from
potentially uninitialized local variables:
src\third_party\skia\src\gpu\grclipmaskmanager.cpp(290) :
warning C6001: Using uninitialized memory 'requiresAA'.
src\third_party\skia\src\gpu\grclipmaskmanager.cpp(336) :
warning C6001: Using uninitialized memory 'genID'.
It is not clear whether the uninitialized reads can actually happen,
but the guarantees seem sufficiently non-obvious that the prudent thing
to do to guarantee future stability is to initialized them -- it's
cheap. I also initialized initialState because it seemed to fall in
the same class, despite there being no warning for it.
BUG=427616
Review URL: https://codereview.chromium.org/957133002
Diffstat (limited to 'src/utils/mac/SkCreateCGImageRef.cpp')
0 files changed, 0 insertions, 0 deletions