englebass pushed a commit to branch master.

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

commit f4c71fba215889e13e77a5b722fca5118a3219c5
Author: Sebastian Dransfeld <[email protected]>
Date:   Sat Dec 7 17:56:55 2013 +0100

    ecore_x: supported isn't an optional argument
    
    if supported is NULL, this function isn't of any use.
    
    Fixes CID 1039430
---
 src/lib/ecore_x/xlib/ecore_x_netwm.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/lib/ecore_x/xlib/ecore_x_netwm.c 
b/src/lib/ecore_x/xlib/ecore_x_netwm.c
index b15832d..83615e7 100644
--- a/src/lib/ecore_x/xlib/ecore_x_netwm.c
+++ b/src/lib/ecore_x/xlib/ecore_x_netwm.c
@@ -137,12 +137,11 @@ ecore_x_netwm_supported_get(Ecore_X_Window root,
 {
    int num_ret;
 
+   EINA_SAFETY_ON_NULL_RETURN_VAL(supported, EINA_FALSE);
+
    if (num)
      *num = 0;
 
-   if (supported)
-     *supported = NULL;
-
    LOGFN(__FILE__, __LINE__, __FUNCTION__);
    num_ret = ecore_x_window_prop_atom_list_get(root, 
ECORE_X_ATOM_NET_SUPPORTED,
                                                supported);

-- 


Reply via email to