Merge "Check for parse error before using the json document"

Juniper · Mar 7, 2016 · 37b2721 · 37b2721
2 parents acf4c45 + 4107e51
commit 37b2721
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/query_engine/select.cc b/src/query_engine/select.cc
@@ -443,7 +443,12 @@ query_status_t SelectQuery::process_query() {
 
             //uint64_t thenj = UTCTimestampUsec();
             rapidjson::Document d;
-            d.Parse<0>(const_cast<char *>(json_string.c_str()));
+            if (d.Parse<0>(const_cast<char *>(
+                    json_string.c_str())).HasParseError()) {
+                QE_LOG(ERROR, "Error parsing json document: " <<
+                       d.GetParseError() << " - " << json_string);
+                continue;
+            }
             //jsont += UTCTimestampUsec() - thenj;
 
             std::vector<StatsSelect::StatEntry> attribs;