diff options
author | Carl-Anton Ingmarsson <ca.ingmarsson@gmail.com> | 2010-07-27 23:17:09 +0200 |
---|---|---|
committer | Carl-Anton Ingmarsson <ca.ingmarsson@gmail.com> | 2010-07-30 11:31:59 +0200 |
commit | d564e1bf19cbeb4bd5aae9cbf92b79445fde96c8 (patch) | |
tree | 4a4033e3ec7f837b59620a34a33750fcc4f566f0 | |
parent | 86860beeaecb747b98014ac5eb6d89c64ac85f60 (diff) |
vdpau: fix GstBaseVideoDecoder returning wrong GstStateChangeReturn
when gst_base_video_decoder_[start|stop] fails we now return
GST_STATE_CHANGE_FAILURE instead of returning what the parent class returns
-rw-r--r-- | sys/vdpau/basevideodecoder/gstbasevideodecoder.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/sys/vdpau/basevideodecoder/gstbasevideodecoder.c b/sys/vdpau/basevideodecoder/gstbasevideodecoder.c index 353badffa..1e9cf14ec 100644 --- a/sys/vdpau/basevideodecoder/gstbasevideodecoder.c +++ b/sys/vdpau/basevideodecoder/gstbasevideodecoder.c | |||
@@ -917,7 +917,8 @@ gst_base_video_decoder_change_state (GstElement * element, | |||
917 | 917 | ||
918 | switch (transition) { | 918 | switch (transition) { |
919 | case GST_STATE_CHANGE_READY_TO_PAUSED: | 919 | case GST_STATE_CHANGE_READY_TO_PAUSED: |
920 | gst_base_video_decoder_start (base_video_decoder); | 920 | if (!gst_base_video_decoder_start (base_video_decoder)) |
921 | return GST_STATE_CHANGE_FAILURE; | ||
921 | break; | 922 | break; |
922 | 923 | ||
923 | default: | 924 | default: |
@@ -928,7 +929,8 @@ gst_base_video_decoder_change_state (GstElement * element, | |||
928 | 929 | ||
929 | switch (transition) { | 930 | switch (transition) { |
930 | case GST_STATE_CHANGE_PAUSED_TO_READY: | 931 | case GST_STATE_CHANGE_PAUSED_TO_READY: |
931 | gst_base_video_decoder_stop (base_video_decoder); | 932 | if (!gst_base_video_decoder_stop (base_video_decoder)) |
933 | ret = GST_STATE_CHANGE_FAILURE; | ||
932 | break; | 934 | break; |
933 | 935 | ||
934 | default: | 936 | default: |