From ec8359fe5415a9cc71a0468873f5a427962fee9c Mon Sep 17 00:00:00 2001 From: Mark Nauwelaerts Date: Mon, 5 Jan 2009 10:28:58 +0000 Subject: [PATCH] [MOVED FROM BAD 21/28] ext/x264/gstx264enc.c: Use hyphen in property name, perform safety buffer size check prior to mem access, and some mo... Original commit message from CVS: * ext/x264/gstx264enc.c: (gst_x264_enc_class_init), (gst_x264_enc_header_buf), (gst_x264_enc_encode_frame): Use hyphen in property name, perform safety buffer size check prior to mem access, and some more parentheses in macro. --- ext/x264/gstx264enc.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/ext/x264/gstx264enc.c b/ext/x264/gstx264enc.c index 21f7b0d..2a77763 100644 --- a/ext/x264/gstx264enc.c +++ b/ext/x264/gstx264enc.c @@ -334,7 +334,7 @@ gst_x264_enc_class_init (GstX264EncClass * klass) "Enable trellis searched quantization", ARG_TRELLIS_DEFAULT, G_PARAM_READWRITE)); g_object_class_install_property (gobject_class, ARG_KEYINT_MAX, - g_param_spec_uint ("key_int_max", "Key-frame maximal interval", + g_param_spec_uint ("key-int-max", "Key-frame maximal interval", "Maximal distance between two key-frames (0 for automatic)", 0, G_MAXINT, ARG_KEYINT_MAX_DEFAULT, G_PARAM_READWRITE)); g_object_class_install_property (gobject_class, ARG_CABAC, @@ -667,8 +667,9 @@ gst_x264_enc_header_buf (GstX264Enc * encoder) /* x264 is expected to return an SEI (some identification info), * followed by an SPS and PPS */ - if (i_nal != 3 || nal[1].i_type != 7 || nal[2].i_type != 8) { - GST_ELEMENT_ERROR (encoder, STREAM, ENCODE, NULL, + if (i_nal != 3 || nal[1].i_type != 7 || nal[2].i_type != 8 || + nal[1].i_payload < 4 || nal[2].i_payload < 1) { + GST_ELEMENT_ERROR (encoder, STREAM, ENCODE, (NULL), ("Unexpected x264 header.")); return NULL; } @@ -941,7 +942,7 @@ gst_x264_enc_encode_frame (GstX264Enc * encoder, x264_picture_t * pic_in, duration = GST_BUFFER_DURATION (in_buf); gst_buffer_unref (in_buf); } else { - GST_ELEMENT_ERROR (encoder, STREAM, ENCODE, NULL, + GST_ELEMENT_ERROR (encoder, STREAM, ENCODE, (NULL), ("Timestamp queue empty.")); return GST_FLOW_ERROR; } -- 2.7.4