devilhorns pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=c45da5c138dca2b9ff26143572c8aee76736a9e7

commit c45da5c138dca2b9ff26143572c8aee76736a9e7
Author: Chris Michael <cp.mich...@samsung.com>
Date:   Wed Jul 12 11:02:16 2017 -0400

    elput: Fix resource leak
    
    Coverity CID1377539 says that we leak the eldbus reply here when we
    error out, so be sure to cleanup 'reply'
    
    Fixes CID1377539
    
    @fix
    
    Signed-off-by: Chris Michael <cp.mich...@samsung.com>
---
 src/lib/elput/elput_logind.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/lib/elput/elput_logind.c b/src/lib/elput/elput_logind.c
index e93633edc1..aa9006e5bb 100644
--- a/src/lib/elput/elput_logind.c
+++ b/src/lib/elput/elput_logind.c
@@ -196,7 +196,10 @@ _logind_session_object_path_get(Elput_Manager *em)
         goto message_fail;
      }
    if (!eldbus_message_arguments_get(reply, "o", &str))
+     {
+        eldbus_message_unref(reply);
         goto message_fail;
+     }
 
    em->dbus.path = strdup(str);
    eldbus_message_unref(reply);

-- 


Reply via email to