diff options
| author | Chris Michael <cp.michael@samsung.com> | 2014-12-10 13:24:54 -0500 |
|---|---|---|
| committer | Chris Michael <cp.michael@samsung.com> | 2014-12-10 13:24:54 -0500 |
| commit | 6e46642dfb8124b34c8733deae4b8d029e362f09 (patch) | |
| tree | 870f3acd384f5542b57f52b8fbe2e50b8439b499 /src/lib/eldbus | |
| parent | 6972d25dc9ac5233f75e8d9b3b96eb5ce35427a9 (diff) | |
| download | efl-6e46642dfb8124b34c8733deae4b8d029e362f09.tar.gz | |
eldbus: Fix possible null dereference reported by Coverity
Summary: Fix Coverity CID1256952: reports a null derefence here due to
eldbus_message_new returning NULL, thus causing a null dereference
when trying to set reply->dbus_msg
@fix
Signed-off-by: Chris Michael <cp.michael@samsung.com>
Diffstat (limited to 'src/lib/eldbus')
| -rw-r--r-- | src/lib/eldbus/eldbus_message.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/lib/eldbus/eldbus_message.c b/src/lib/eldbus/eldbus_message.c index d6b18d3596..03b8d642c2 100644 --- a/src/lib/eldbus/eldbus_message.c +++ b/src/lib/eldbus/eldbus_message.c @@ -835,6 +835,8 @@ eldbus_message_error_new(const Eldbus_Message *msg, const char *error_name, cons EINA_SAFETY_ON_NULL_RETURN_VAL(error_msg, NULL); reply = eldbus_message_new(EINA_FALSE); + EINA_SAFETY_ON_NULL_RETURN_VAL(reply, NULL); + reply->dbus_msg = dbus_message_new_error(msg->dbus_msg, error_name, error_msg); |