From c3b7b9cd39a01cf297839fabdc10f63a2f2ffcc6 Mon Sep 17 00:00:00 2001 From: "Xiang, Haihao" Date: Fri, 26 Jan 2018 15:35:22 +0800 Subject: Avoid dereferencing a pointer before checking pointer against NULL It is possible a sequence parameter buffer is not provided by application. Signed-off-by: Xiang, Haihao --- src/i965_encoder.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/i965_encoder.c b/src/i965_encoder.c index 04f108eb..d831541d 100644 --- a/src/i965_encoder.c +++ b/src/i965_encoder.c @@ -1024,12 +1024,11 @@ intel_encoder_check_mpeg2_parameter(VADriverContextP ctx, { struct i965_driver_data *i965 = i965_driver_data(ctx); VAEncPictureParameterBufferMPEG2 *pic_param = (VAEncPictureParameterBufferMPEG2 *)encode_state->pic_param_ext->buffer; - VAEncSequenceParameterBufferMPEG2 *seq_param = (VAEncSequenceParameterBufferMPEG2 *)encode_state->seq_param_ext->buffer; + VAEncSequenceParameterBufferMPEG2 *seq_param = NULL; struct object_surface *obj_surface; struct object_buffer *obj_buffer; int i = 0; - seq_param = NULL; if (encode_state->seq_param_ext && encode_state->seq_param_ext->buffer) seq_param = (VAEncSequenceParameterBufferMPEG2 *)encode_state->seq_param_ext->buffer; -- cgit v1.2.1