diff options
author | Alessandro Decina <alessandro@nnva.org> | 2006-08-25 09:42:43 +0000 |
---|---|---|
committer | Wim Taymans <wim.taymans@gmail.com> | 2006-08-25 09:42:43 +0000 |
commit | 2f4517a70b9ef85bd18d232b6596848f8a285f7c (patch) | |
tree | 2f9e807b42c9ab22328452db45a03fe0930dc91d /ext/annodex/gstcmmlenc.c | |
parent | 2019f527f701018622083cdbba4b12b4f53875de (diff) |
ext/annodex/gstannodex.c: Do some extra sanity checks.
Original commit message from CVS:
Patch by: Alessandro Decina <alessandro at nnva dot org>
* ext/annodex/gstannodex.c: (gst_annodex_granule_to_time):
Do some extra sanity checks.
Fixes #350340.
* ext/annodex/gstcmmlenc.c: (gst_cmml_enc_change_state),
(gst_cmml_enc_parse_tag_head), (gst_cmml_enc_parse_tag_clip),
(gst_cmml_enc_push_clip), (gst_cmml_enc_push):
Check if clip->start_time is valid before adding the clip to the
track list.
Reset enc->preamble going from PAUSED to READY.
Don't use GST_FLOW_UNEXPECTED for wrong usage of the element, it is
only used for EOS.
Only post an error message if we were the one that created the fatal
GstFlowReturn value.
* ext/annodex/gstcmmlutils.c: (gst_cmml_clock_time_from_npt),
(gst_cmml_clock_time_to_granule), (gst_cmml_track_list_has_clip):
Parse the seconds field of the npt-sec time format using %llu rather than
%d and check that the value scaled by GST_SECOND doesn't overflow.
Use guint64(s) to represent the keyindex and keyoffset fields of a granulepos.
Lookup a clip's track with clip->track rather than clip->id which
makes no sense.
Identify a clip by its track and start time and not its xml id.
do some more input checking and make sure we don't do undefined shifts.
* tests/check/elements/cmmldec.c: (setup_cmmldec),
(teardown_cmmldec), (check_output_buffer_is_equal), (push_data),
(cmml_tag_message_pop), (check_headers), (push_clip_full),
(push_clip), (push_empty_clip), (check_output_clip),
(GST_START_TEST), (cmmldec_suite):
* tests/check/elements/cmmlenc.c: (setup_cmmlenc),
(teardown_cmmlenc), (check_output_buffer_is_equal), (push_data),
(check_headers), (push_clip), (check_clip_times), (check_clip),
(check_empty_clip), (GST_START_TEST), (cmmlenc_suite):
Added some more checks.
Diffstat (limited to 'ext/annodex/gstcmmlenc.c')
-rw-r--r-- | ext/annodex/gstcmmlenc.c | 20 |
1 files changed, 11 insertions, 9 deletions
diff --git a/ext/annodex/gstcmmlenc.c b/ext/annodex/gstcmmlenc.c index d8392c63..9b2c03c6 100644 --- a/ext/annodex/gstcmmlenc.c +++ b/ext/annodex/gstcmmlenc.c @@ -259,6 +259,7 @@ gst_cmml_enc_change_state (GstElement * element, GstStateChange transition) gst_cmml_track_list_destroy (enc->tracks); enc->tracks = NULL; g_free (enc->preamble); + enc->preamble = NULL; gst_cmml_parser_free (enc->parser); break; } @@ -468,7 +469,7 @@ gst_cmml_enc_parse_tag_head (GstCmmlEnc * enc, GstCmmlTagHead * head) flow_unexpected: GST_ELEMENT_ERROR (enc, STREAM, ENCODE, (NULL), ("got head tag before preamble")); - enc->flow_return = GST_FLOW_UNEXPECTED; + enc->flow_return = GST_FLOW_ERROR; return; push_error: gst_caps_unref (caps); @@ -507,6 +508,14 @@ gst_cmml_enc_parse_tag_clip (GstCmmlEnc * enc, GstCmmlTagClip * clip) (gchar *) clip->track); if (prev_clip) { prev_clip_time = prev_clip->start_time; + if (prev_clip_time > clip->start_time) { + GST_ELEMENT_ERROR (enc, STREAM, ENCODE, + (NULL), ("previous clip start time > current clip (%s) start time", + clip->id)); + enc->flow_return = GST_FLOW_ERROR; + return; + } + /* we don't need the prev clip anymore */ gst_cmml_track_list_del_clip (enc->tracks, prev_clip); } @@ -526,12 +535,6 @@ gst_cmml_enc_push_clip (GstCmmlEnc * enc, GstCmmlTagClip * clip, gchar *clip_string; gint64 granulepos; - if (prev_clip_time != GST_CLOCK_TIME_NONE && - prev_clip_time > clip->start_time) { - GST_WARNING_OBJECT (enc, - "previous clip start time > current clip (%s) start time", clip->id); - } - /* encode the clip */ clip_string = (gchar *) gst_cmml_parser_tag_clip_to_string (enc->parser, clip); @@ -591,8 +594,7 @@ gst_cmml_enc_push (GstCmmlEnc * enc, GstBuffer * buffer) res = gst_pad_push (enc->srcpad, buffer); if (GST_FLOW_IS_FATAL (res)) - GST_ELEMENT_ERROR (enc, STREAM, ENCODE, - (NULL), ("could not push buffer: %s", gst_flow_get_name (res))); + GST_WARNING_OBJECT (enc, "push returned: %s", gst_flow_get_name (res)); return res; } |