From: Michel Dänzer <[email protected]>

Fixes memory leak when destroying pixmaps with priv->bo == NULL.

Reported-by: Qiang Yu <[email protected]>
Signed-off-by: Michel Dänzer <[email protected]>
---
 src/amdgpu_pixmap.h | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/amdgpu_pixmap.h b/src/amdgpu_pixmap.h
index 9c51067..a8de26a 100644
--- a/src/amdgpu_pixmap.h
+++ b/src/amdgpu_pixmap.h
@@ -63,10 +63,10 @@ static inline Bool amdgpu_set_pixmap_bo(PixmapPtr pPix, 
struct amdgpu_buffer *bo
                return TRUE;
 
        if (priv) {
-               if (priv->bo == bo)
-                       return TRUE;
-
                if (priv->bo) {
+                       if (priv->bo == bo)
+                               return TRUE;
+
                        amdgpu_bo_unref(&priv->bo);
                        priv->handle_valid = FALSE;
                }
-- 
2.8.1

_______________________________________________
xorg-driver-ati mailing list
[email protected]
https://lists.x.org/mailman/listinfo/xorg-driver-ati

Reply via email to