diff options
| author | Xiang, Haihao <haihao.xiang@intel.com> | 2018-01-26 15:35:22 +0800 |
|---|---|---|
| committer | Xiang, Haihao <haihao.xiang@intel.com> | 2018-01-29 09:07:38 +0800 |
| commit | c3b7b9cd39a01cf297839fabdc10f63a2f2ffcc6 (patch) | |
| tree | 05ae46b51f93bdb0ba881cb84356236dcb25a1ef | |
| parent | 0b37282fa1c302d2b0c791fdc46c69af0d16d09b (diff) | |
| download | libva-intel-driver-c3b7b9cd39a01cf297839fabdc10f63a2f2ffcc6.tar.gz | |
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 <haihao.xiang@intel.com>
| -rw-r--r-- | src/i965_encoder.c | 3 |
1 files changed, 1 insertions, 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; |