From b5f61341ddbe3d7b011c01e974d658ec503c795f Mon Sep 17 00:00:00 2001 From: Mark Nauwelaerts Date: Mon, 5 Jan 2009 10:28:58 +0000 Subject: [PATCH] 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. --- ChangeLog | 7 +++++++ ext/x264/gstx264enc.c | 9 +++++---- 2 files changed, 12 insertions(+), 4 deletions(-) diff --git a/ChangeLog b/ChangeLog index f1b31e0..f31c612 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,10 @@ +2009-01-05 Mark Nauwelaerts + + * 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. + 2009-01-05 Sebastian Dröge * gst/mxf/mxfaes-bwf.c: 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