On 07/26/2017 08:14 PM, Michel Dänzer wrote:
From: Michel Dänzer <michel.daen...@amd.com>

Fixes a drawable leak.

Fixes: bbc29393d3be ("st/mesa: create framebuffer iface hash table per
                       st manager")
Tested-by: Nick Sarnie <commendsar...@gmail.com>
Signed-off-by: Michel Dänzer <michel.daen...@amd.com>
---
  src/mesa/state_tracker/st_manager.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mesa/state_tracker/st_manager.c 
b/src/mesa/state_tracker/st_manager.c
index 6447403fc6..97bf89fc65 100644
--- a/src/mesa/state_tracker/st_manager.c
+++ b/src/mesa/state_tracker/st_manager.c
@@ -583,7 +583,7 @@ static void
  st_api_destroy_drawable(struct st_api *stapi,
                          struct st_framebuffer_iface *stfbi)
  {
-   if (stfbi)
+   if (!stfbi)
        return;

     st_framebuffer_iface_remove(stfbi->state_manager, stfbi);


Reviewed-by: Brian Paul <bri...@vmware.com>

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to