On 12/30/2016 04:08 PM, peng.chen wrote:
Signed-off-by: peng.chen<peng.c.c...@intel.com>
In fact the dri_bo_unreference function can handle the scenario that bo is NULL.
Of course it also looks good to me that the explicit check is added in the i965_destroy_surface_storage.
Add: Reviewed-by: Zhao Yakui <yakui.z...@intel.com> Thanks
--- src/i965_drv_video.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/i965_drv_video.c b/src/i965_drv_video.c index bf59374..ded3d93 100644 --- a/src/i965_drv_video.c +++ b/src/i965_drv_video.c @@ -1210,7 +1210,7 @@ VAStatus i965_QueryConfigAttributes(VADriverContextP ctx, void i965_destroy_surface_storage(struct object_surface *obj_surface) { - if (!obj_surface) + if (!obj_surface || !obj_surface->bo) return; dri_bo_unreference(obj_surface->bo);
_______________________________________________ Libva mailing list Libva@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libva