projects / mesa.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 6c731d8) | patch
virgl: wrap ret assignment with braces to do correct thing
authorDave Airlie <airlied@gmail.com>
Sat, 31 Oct 2015 06:19:43 +0000 (16:19 +1000)
committerDave Airlie <airlied@gmail.com>
Sat, 7 Nov 2015 20:27:02 +0000 (06:27 +1000)
commit87711183ac35d85ca7d2c2ee67536fe689d6bef3
tree7c8fb7c681612934638a75db40e8565250709e24tree
parent6c731d85666abb61c49e5b4affa196545f5ac086commit | diff
virgl: wrap ret assignment with braces to do correct thing

Coverity reported that ret could only be 0 or 1, since it
was setting ret = fn() > 0, instead of doing (ret = fn()) > 0.

Signed-off-by: Dave Airlie <airlied@redhat.com>
src/gallium/winsys/virgl/drm/virgl_drm_winsys.c diff | blob | history
src/gallium/winsys/virgl/vtest/virgl_vtest_winsys.c diff | blob | history