From 7edef2f006c58f3dbaa36f88d3922ac3b1fbc447 Mon Sep 17 00:00:00 2001
From: Jacob Keeler <jacob.keeler@livioradio.com>
Date: Tue, 5 Apr 2022 09:29:23 -0400
Subject: Check if JSON value is a map before accessing members (#3896)

---
 src/components/security_manager/src/security_manager_impl.cc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/components/security_manager/src/security_manager_impl.cc b/src/components/security_manager/src/security_manager_impl.cc
index 75cc104d95..59f6da22be 100644
--- a/src/components/security_manager/src/security_manager_impl.cc
+++ b/src/components/security_manager/src/security_manager_impl.cc
@@ -607,10 +607,11 @@ bool SecurityManagerImpl::ProcessInternalError(
   Json::Value root;
   utils::JsonReader reader;
 
-  if (!reader.parse(str, &root)) {
+  if (!reader.parse(str, &root) || !root.isObject()) {
     SDL_LOG_DEBUG("Json parsing fails.");
     return false;
   }
+
   uint8_t id = root[kErrId].asInt();
   SDL_LOG_DEBUG("Received InternalError id " << std::to_string(id) << ", text: "
                                              << root[kErrText].asString());
-- 
cgit v1.2.1