Add assertion after malloc in the message class

Since message class is written under presumtion that malloc will never
return NULL(there is no NULL-checking code), it is better to insert an
assertion after malloc was called.

This solves bug reported by SVACE with WGID:
696636 695764 696678 696677 696676 696638 696635 696654 696653 696652
696651 696650 696649 696648 696647 696646 696643 696641 696640 696639

Change-Id: I258a1d141a18283d9b5a5108e82f505ba610c86d
Signed-off-by: SangYoun Kwak <sy.kwak@samsung.com>

diff --git a/src/shared/message.cpp b/src/shared/message.cpp
index 5f12d23..f8ab962 100644 (file)
--- a/src/shared/message.cpp
+++ b/src/shared/message.cpp
@@ -22,6 +22,8 @@
 #include <sensor_log.h>
 #include <atomic>
 
+#include <assert.h>
+
 using namespace ipc;
 
 #define UNDEFINED_TYPE -2
@@ -33,6 +35,7 @@ message::message(size_t capacity)
        , m_capacity(capacity)
        , m_msg((char *)malloc(sizeof(char) * capacity))
 {
+       assert(m_msg != NULL);
        m_header.id = sequence++;
        m_header.type = UNDEFINED_TYPE;
        m_header.length = m_size;
@@ -55,6 +58,7 @@ message::message(const message &msg)
        , m_capacity(msg.m_capacity)
        , m_msg((char *)malloc(sizeof(char) * msg.m_capacity))
 {
+       assert(m_msg != NULL);
        ::memcpy(&m_header, &msg.m_header, sizeof(message_header));
        ::memcpy(m_msg, msg.m_msg, msg.m_size);
 }