raster pushed a commit to branch master.

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

commit e263af08c9269eef66141fbca0e8a1a0cd6b8d2a
Author: Carsten Haitzler (Rasterman) <[email protected]>
Date:   Thu Aug 14 18:52:31 2014 +0900

    eldbus - fix varargs missing end in iter get and next
    
    there was an error return handler that would return without doing
    va_end(). coverity picked this up. this fixes CID 1039883
---
 src/lib/eldbus/eldbus_message.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/lib/eldbus/eldbus_message.c b/src/lib/eldbus/eldbus_message.c
index 1d1eeb8..aa44410 100644
--- a/src/lib/eldbus/eldbus_message.c
+++ b/src/lib/eldbus/eldbus_message.c
@@ -717,7 +717,11 @@ eldbus_message_iter_get_and_next(Eldbus_Message_Iter 
*iter, char signature, ...)
         Eldbus_Message_Iter **iter_var = va_arg(vl, Eldbus_Message_Iter**);
 
         sub = _message_iterator_new(EINA_FALSE);
-        EINA_SAFETY_ON_NULL_RETURN_VAL(sub, EINA_FALSE);
+        if (!sub)
+          {
+             va_end(vl);
+             return EINA_FALSE;
+          }
         dbus_message_iter_recurse(&iter->dbus_iterator,
                                   &sub->dbus_iterator);
         iter->iterators = eina_inlist_append(iter->iterators,

-- 


Reply via email to