diff options
| author | Tom Sepez <tsepez@chromium.org> | 2014-10-14 14:40:57 -0700 |
|---|---|---|
| committer | Tom Sepez <tsepez@chromium.org> | 2014-10-14 14:40:57 -0700 |
| commit | 35d720aff01c5ea778c16ac1e31c56f68490f10b (patch) | |
| tree | bdc1fcfb0ff5bc9d48ab3fde4f996bd3fe4b6ecb /core/include | |
| parent | 9776eca31c69dfa154e20b784a9df396b134758f (diff) | |
| download | pdfium-35d720aff01c5ea778c16ac1e31c56f68490f10b.tar.xz | |
Don't leave dangling pointer to out-of-scope local in CPDF_StreamContentParser::Parse.
This is just a bit of defensive programming; I'm not sure the situation can
occur in the current code, but the following code is likely to set off a red
flag to anyone who reads it:
CPDF_StreamParser syntax(pData, dwSize);
m_pSyntax = &syntax;
since the extent of the local |syntax| is far less than the pointer member
|m_pSyntax|. NULL it out before syntax goes out of scope.
R=jun_fang@foxitsoftware.com
Review URL: https://codereview.chromium.org/652063002
Diffstat (limited to 'core/include')
0 files changed, 0 insertions, 0 deletions