From: Christophe CURIS <[email protected]> As pointed by Coverity, there was a check for null pointer on scr->dock_dots, but this check was not made in the 2nd use of it, done if the HIDDENDOT feature was enabled.
Investigation show that it is not possible that this pointer could be created NULL, so let's remove the unneeded check. Signed-off-by: Christophe CURIS <[email protected]> --- src/appicon.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/appicon.c b/src/appicon.c index 524fded..1d28a24 100644 --- a/src/appicon.c +++ b/src/appicon.c @@ -411,7 +411,7 @@ void wAppIconPaint(WAppIcon *aicon) if (aicon->docked && scr->dock && scr->dock == aicon->dock && aicon->yindex == 0) updateDockNumbers(scr); # endif - if (scr->dock_dots && aicon->docked && !aicon->running && aicon->command != NULL) { + if (aicon->docked && !aicon->running && aicon->command != NULL) { XSetClipMask(dpy, scr->copy_gc, scr->dock_dots->mask); XSetClipOrigin(dpy, scr->copy_gc, 0, 0); XCopyArea(dpy, scr->dock_dots->image, aicon->icon->core->window, -- 2.1.1 -- To unsubscribe, send mail to [email protected].
