summaryrefslogtreecommitdiff
path: root/core/fxcrt
diff options
context:
space:
mode:
authorMiklos Vajna <vmiklos@vmiklos.hu>2017-02-25 14:44:57 +0100
committerChromium commit bot <commit-bot@chromium.org>2017-02-26 00:00:45 +0000
commitf2225817ddbe05e2caa751da23c4a6d44c5160bd (patch)
tree86ee9b8c0485198cd84dc9fcdf9e199702045a4f /core/fxcrt
parent412fa65331cc04776bf1e3da5f51f29ea0fc7937 (diff)
downloadpdfium-f2225817ddbe05e2caa751da23c4a6d44c5160bd.tar.xz
core: fix two more uninitialized memberschromium/3025chromium/3024
Found by Coverity when scanning the bundled pdfium-3004 in LibreOffice. This fixes: - CID 1400335: Uninitialized members (UNINIT_CTOR) CPDF_DataAvail::m_dwTrailerOffset - CID 1400334: Uninitialized members (UNINIT_CTOR) CXML_Element::m_pParent Change-Id: I42da9a704ad75adf39c510c6ef89eb7a97860ea1 Reviewed-on: https://pdfium-review.googlesource.com/2670 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org>
Diffstat (limited to 'core/fxcrt')
-rw-r--r--core/fxcrt/fx_xml.h2
-rw-r--r--core/fxcrt/fx_xml_parser.cpp13
2 files changed, 2 insertions, 13 deletions
diff --git a/core/fxcrt/fx_xml.h b/core/fxcrt/fx_xml.h
index 7f42a7fa9b..38a1b60846 100644
--- a/core/fxcrt/fx_xml.h
+++ b/core/fxcrt/fx_xml.h
@@ -66,8 +66,6 @@ class CXML_Element {
bool bSaveSpaceChars = false,
FX_FILESIZE* pParsedSize = nullptr);
- CXML_Element(const CFX_ByteStringC& qSpace, const CFX_ByteStringC& tagName);
- explicit CXML_Element(const CFX_ByteStringC& qTagName);
CXML_Element();
~CXML_Element();
diff --git a/core/fxcrt/fx_xml_parser.cpp b/core/fxcrt/fx_xml_parser.cpp
index 7658620424..2250087b61 100644
--- a/core/fxcrt/fx_xml_parser.cpp
+++ b/core/fxcrt/fx_xml_parser.cpp
@@ -715,17 +715,8 @@ CXML_Element* CXML_Element::Parse(
return XML_ContinueParse(parser, bSaveSpaceChars, pParsedSize);
}
-CXML_Element::CXML_Element() : m_QSpaceName(), m_TagName(), m_AttrMap() {}
-CXML_Element::CXML_Element(const CFX_ByteStringC& qSpace,
- const CFX_ByteStringC& tagName)
- : m_QSpaceName(), m_TagName(), m_AttrMap() {
- m_QSpaceName = qSpace;
- m_TagName = tagName;
-}
-CXML_Element::CXML_Element(const CFX_ByteStringC& qTagName)
- : m_pParent(nullptr), m_QSpaceName(), m_TagName(), m_AttrMap() {
- SetTag(qTagName);
-}
+CXML_Element::CXML_Element()
+ : m_pParent(nullptr), m_QSpaceName(), m_TagName(), m_AttrMap() {}
CXML_Element::~CXML_Element() {
Empty();
}