devilhorns pushed a commit to branch master.

commit 2047bac4b0959cfedb4c080a10ac5369b648a94a
Author: Chris Michael <[email protected]>
Date:   Fri Jun 28 14:44:25 2013 +0100

    Fix klockwork issue: NULL pointer may be dereferenced:
    Don't call e_comp_win_effect_set unless we have a valid pager window
    
    Signed-off-by: Chris Michael <[email protected]>
---
 src/modules/pager/e_mod_main.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/modules/pager/e_mod_main.c b/src/modules/pager/e_mod_main.c
index 3c47f07..b7c8bca 100644
--- a/src/modules/pager/e_mod_main.c
+++ b/src/modules/pager/e_mod_main.c
@@ -2196,12 +2196,12 @@ _pager_drop_cb_drop(void *data, const char *type, void 
*event_info)
                e_border_hide(bd, 2);
              else
                {
-                  if (pd->desk != bd->desk)
+                  if ((pd->desk != bd->desk) && (pw))
                     e_comp_win_effect_set(pw->border->cw, NULL);
                }
              e_border_desk_set(bd, pd->desk);
              e_border_raise(bd);
-                  
+
              if ((!max) && (!fullscreen))
                {
                   int zx, zy, zw, zh, mx, my;

-- 

------------------------------------------------------------------------------
This SF.net email is sponsored by Windows:

Build for Windows Store.

http://p.sf.net/sfu/windows-dev2dev

Reply via email to