diff options
author | Caolán McNamara <caolanm@redhat.com> | 2017-08-03 14:37:30 +0100 |
---|---|---|
committer | Michael Stahl <mstahl@redhat.com> | 2017-08-07 22:30:39 +0200 |
commit | d269f530cc834507f205374727c20cc5992e2213 (patch) | |
tree | 23a181a7ad09169ef4ba93738a4fd4c05d1a75ed | |
parent | c9f07d72b6e40ca70ee11f0350fd6e5acfefba89 (diff) |
ofz#2846 null deref
Change-Id: I88b61d7a4faaed118db8df6f99cef08310c1f2eb
Reviewed-on: https://gerrit.libreoffice.org/40725
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
-rw-r--r-- | hwpfilter/source/hwpread.cxx | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/hwpfilter/source/hwpread.cxx b/hwpfilter/source/hwpread.cxx index 90564df487b9..74edfda45845 100644 --- a/hwpfilter/source/hwpread.cxx +++ b/hwpfilter/source/hwpread.cxx @@ -428,6 +428,13 @@ bool Picture::Read(HWPFile & hwpf) UpdateBBox(this); if( pictype != PICTYPE_DRAW ) style.cell = reserved3; + else + { + //picinfo.picun read above is unioned with + //picinfo.picdraw and so wrote to the hdo pointer + //value, which is definitely not useful to us + picinfo.picdraw.hdo = nullptr; + } if (follow_block_size != 0) { |