[Mesa-dev] [PATCH] postprocess: Check ppq is null before calling pp_free_bos.

Vinson Lee vlee at freedesktop.org
Thu Jun 27 22:15:39 PDT 2013


pp_free_bos dereferences ppq without a null check.

Fixes "Dereference before null check" defect reported by Coverity.

Signed-off-by: Vinson Lee <vlee at freedesktop.org>
---
 src/gallium/auxiliary/postprocess/pp_init.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/gallium/auxiliary/postprocess/pp_init.c b/src/gallium/auxiliary/postprocess/pp_init.c
index 1130248..b4eb204 100644
--- a/src/gallium/auxiliary/postprocess/pp_init.c
+++ b/src/gallium/auxiliary/postprocess/pp_init.c
@@ -177,7 +177,9 @@ pp_free(struct pp_queue_t *ppq)
 {
    unsigned int i, j;
 
-   pp_free_fbos(ppq);
+   if (ppq) {
+      pp_free_fbos(ppq);
+   }
 
    if (ppq && ppq->p) {
       /* Only destroy created contexts. */
-- 
1.8.2.1



More information about the mesa-dev mailing list