On 09/29/2015 05:30 PM, Lim Siew Hoon wrote:
Signed-off-by: Lim Siew Hoon<[email protected]>
Thanks for the patch.
It is oK to me that more check is added for the obj_context.
In fact this is from the upper caller and it is already checked in the
caller. (If it is NULL, it will fail in the upper caller).
Of course it is still OK to me that this check is added.
Thanks
Yakui
---
src/i965_drv_video.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/i965_drv_video.c b/src/i965_drv_video.c
index 9bbd67f..cb950e1 100644
--- a/src/i965_drv_video.c
+++ b/src/i965_drv_video.c
@@ -2813,6 +2813,9 @@ i965_decoder_wrapper_picture(VADriverContextP ctx,
VADriverContextP pdrvctx;
struct object_buffer *obj_buffer;
+ if (obj_context == NULL)
+ return VA_STATUS_ERROR_INVALID_CONTEXT;
+
/* When it is not wrapped context, continue the normal flowchart */
if (obj_context->wrapper_context == VA_INVALID_ID)
return vaStatus;
_______________________________________________
Libva mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libva
