Swift
summaryrefslogtreecommitdiffstats
path: root/Swiften/Parser/LibXMLParser.cpp
diff options
context:
space:
mode:
Diffstat (limited to 'Swiften/Parser/LibXMLParser.cpp')
-rw-r--r--Swiften/Parser/LibXMLParser.cpp16
1 files changed, 10 insertions, 6 deletions
diff --git a/Swiften/Parser/LibXMLParser.cpp b/Swiften/Parser/LibXMLParser.cpp
index be0a92d..5bd3737 100644
--- a/Swiften/Parser/LibXMLParser.cpp
+++ b/Swiften/Parser/LibXMLParser.cpp
@@ -1,5 +1,5 @@
1/* 1/*
2 * Copyright (c) 2010-2016 Isode Limited. 2 * Copyright (c) 2010-2018 Isode Limited.
3 * All rights reserved. 3 * All rights reserved.
4 * See the COPYING file for more information. 4 * See the COPYING file for more information.
5 */ 5 */
@@ -8,11 +8,10 @@
8 8
9#include <cassert> 9#include <cassert>
10#include <cstring> 10#include <cstring>
11#include <limits>
11#include <memory> 12#include <memory>
12#include <string> 13#include <string>
13 14
14#include <boost/numeric/conversion/cast.hpp>
15
16#include <libxml/parser.h> 15#include <libxml/parser.h>
17 16
18#include <Swiften/Base/Log.h> 17#include <Swiften/Base/Log.h>
@@ -36,11 +35,12 @@ static void handleStartElement(void* parser, const xmlChar* name, const xmlChar*
36 if (attributes[i+2]) { 35 if (attributes[i+2]) {
37 attributeNS = std::string(reinterpret_cast<const char*>(attributes[i+2])); 36 attributeNS = std::string(reinterpret_cast<const char*>(attributes[i+2]));
38 } 37 }
38 assert(attributes[i+4] >= attributes[i+3]);
39 attributeValues.addAttribute( 39 attributeValues.addAttribute(
40 std::string(reinterpret_cast<const char*>(attributes[i])), 40 std::string(reinterpret_cast<const char*>(attributes[i])),
41 attributeNS, 41 attributeNS,
42 std::string(reinterpret_cast<const char*>(attributes[i+3]), 42 std::string(reinterpret_cast<const char*>(attributes[i+3]),
43 boost::numeric_cast<size_t>(attributes[i+4]-attributes[i+3]))); 43 static_cast<size_t>(attributes[i+4]-attributes[i+3])));
44 } 44 }
45 static_cast<XMLParser*>(parser)->getClient()->handleStartElement(reinterpret_cast<const char*>(name), (xmlns ? reinterpret_cast<const char*>(xmlns) : std::string()), attributeValues); 45 static_cast<XMLParser*>(parser)->getClient()->handleStartElement(reinterpret_cast<const char*>(name), (xmlns ? reinterpret_cast<const char*>(xmlns) : std::string()), attributeValues);
46} 46}
@@ -50,7 +50,8 @@ static void handleEndElement(void *parser, const xmlChar* name, const xmlChar*,
50} 50}
51 51
52static void handleCharacterData(void* parser, const xmlChar* data, int len) { 52static void handleCharacterData(void* parser, const xmlChar* data, int len) {
53 static_cast<XMLParser*>(parser)->getClient()->handleCharacterData(std::string(reinterpret_cast<const char*>(data), boost::numeric_cast<size_t>(len))); 53 assert(len >= 0);
54 static_cast<XMLParser*>(parser)->getClient()->handleCharacterData(std::string(reinterpret_cast<const char*>(data), static_cast<size_t>(len)));
54} 55}
55 56
56static void handleError(void*, const char* /*m*/, ... ) { 57static void handleError(void*, const char* /*m*/, ... ) {
@@ -94,7 +95,10 @@ LibXMLParser::~LibXMLParser() {
94} 95}
95 96
96bool LibXMLParser::parse(const std::string& data) { 97bool LibXMLParser::parse(const std::string& data) {
97 if (xmlParseChunk(p->context_, data.c_str(), boost::numeric_cast<int>(data.size()), false) == XML_ERR_OK) { 98 if (data.size() > std::numeric_limits<int>::max()) {
99 return false;
100 }
101 if (xmlParseChunk(p->context_, data.c_str(), static_cast<int>(data.size()), false) == XML_ERR_OK) {
98 return true; 102 return true;
99 } 103 }
100 xmlError* error = xmlCtxtGetLastError(p->context_); 104 xmlError* error = xmlCtxtGetLastError(p->context_);