diff options
author | Tom Sepez <tsepez@chromium.org> | 2014-11-06 15:00:36 -0800 |
---|---|---|
committer | Tom Sepez <tsepez@chromium.org> | 2014-11-06 15:00:36 -0800 |
commit | 09a22a6cd9758e630202d1730aaa8fd7898c91cb (patch) | |
tree | 4cc9b016ee1b8743889728371dd36a22a7b9ea38 /core | |
parent | d9ecee1065cf391979dee0051b53fb5ff729ca2e (diff) | |
download | pdfium-09a22a6cd9758e630202d1730aaa8fd7898c91cb.tar.xz |
Fix bug with reading from uninitialized variable found by VC++'s /analyze.
The flag variable is conditionally initialized but unconditionally read.
Warning was:
src\fpdfapi\fpdf_page\fpdf_page_pattern.cpp(274) : warning C6001:
Using uninitialized memory 'flag'.
BUG=427616
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/703213004
Diffstat (limited to 'core')
-rw-r--r-- | core/src/fpdfapi/fpdf_page/fpdf_page_pattern.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/core/src/fpdfapi/fpdf_page/fpdf_page_pattern.cpp b/core/src/fpdfapi/fpdf_page/fpdf_page_pattern.cpp index 467ef9b3f6..4644302c5b 100644 --- a/core/src/fpdfapi/fpdf_page/fpdf_page_pattern.cpp +++ b/core/src/fpdfapi/fpdf_page/fpdf_page_pattern.cpp @@ -266,7 +266,7 @@ CFX_FloatRect _GetShadingBBox(CPDF_Stream* pStream, int type, const CFX_AffineMa int full_point_count = type == 7 ? 16 : (type == 6 ? 12 : 1); int full_color_count = (type == 6 || type == 7) ? 4 : 1; while (!stream.m_BitStream.IsEOF()) { - FX_DWORD flag; + FX_DWORD flag = 0; if (type != 5) { flag = stream.GetFlag(); } |