Modified for Svace Issue (NONTERMINATED_STRING) 27/123627/1
authorSegwon <segwon.han@samsung.com>
Thu, 6 Apr 2017 08:48:57 +0000 (17:48 +0900)
committerSegwon <segwon.han@samsung.com>
Thu, 6 Apr 2017 08:54:18 +0000 (17:54 +0900)
Issue : The Json structure, which is open source, does memcpy when it inserts string data, but there is no code to calculate the size when a char pointer is inserted.

Signed-off-by: Segwon <segwon.han@samsung.com>
Change-Id: If8487ad732cf279d08a73f3d6572bc8f0cd7fef0

msf_tizen_client/src/Channel.cpp

index da5e4ee..56b3a13 100755 (executable)
@@ -1152,7 +1152,8 @@ void Channel::publish(string event, const char *data, list<Client> clients,
        Json::Value to_list;
        std::list<Client>::iterator iterator;
        for (iterator = clients.begin(); iterator != clients.end(); ++iterator) {
-               to_list.append(iterator->getId());
+               string id = iterator->getId();
+               to_list.append(id);
        }
        // TODO
        publishMessage(event, data, to_list, payload, payload_size, user_data);
@@ -1219,7 +1220,8 @@ unsigned char *Channel::prepareMessageMap(string method, string event,
 
        params["event"] = event;
        if (data) {
-               params["data"] = data;
+               string str_data = data;
+               params["data"] = str_data;
        }
        params["to"] = to;
        message["method"] = method;