diff options
| author | Adam Jackson <ajax@nwnk.net> | 2006-03-15 16:01:47 +0000 |
|---|---|---|
| committer | Adam Jackson <ajax@nwnk.net> | 2006-03-15 16:01:47 +0000 |
| commit | 6bb2dc02a7cffd6ed7dd28e88d584920a4150749 (patch) | |
| tree | 4278f359ab894257c03a374f17a8751162edf566 | |
| parent | 1e5c0842af99027cc6c30a16f967d8b60c9a894d (diff) | |
Coverity #337: Remove useless NULL check.
| -rw-r--r-- | ChangeLog | 3 | ||||
| -rw-r--r-- | miext/cw/cw.c | 2 |
2 files changed, 4 insertions, 1 deletions
@@ -3,6 +3,9 @@ * hw/xfree86/loader/loadmod.c: Coverity #1053: Nuke a dead variable. + * miext/cw/cw.c: + Coverity #337: Remove useless NULL check. + 2006-03-15 Benjamin Herrenschmidt <benh@kernel.crashing.org> * hw/xfree86/dri/dri.c: (DRIExtensionInit): diff --git a/miext/cw/cw.c b/miext/cw/cw.c index 9c2ea9677..f60f8cf28 100644 --- a/miext/cw/cw.c +++ b/miext/cw/cw.c @@ -463,7 +463,7 @@ cwPaintWindowBackground(WindowPtr pWin, RegionPtr pRegion, int what) DrawablePtr pBackingDrawable; int x_off, y_off, x_screen, y_screen; - while (pWin && pWin->backgroundState == ParentRelative) + while (pWin->backgroundState == ParentRelative) pWin = pWin->parent; pBackingDrawable = cwGetBackingDrawable((DrawablePtr)pWin, &x_off, |