From 612cdeec80da95b12f9fad928419c7de2be20d68 Mon Sep 17 00:00:00 2001 From: Vincent Penquerc'h Date: Wed, 9 Apr 2014 14:37:48 +0100 Subject: [PATCH] resindvd: avoid crashing in pathological case When we'd see an unknown stream type, then a SDDS stream. Then we'd get to the end of the switch with a NULL temp stream pointer, and dereference it. Coverity 1139708 --- ext/resindvd/gstmpegdemux.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/ext/resindvd/gstmpegdemux.c b/ext/resindvd/gstmpegdemux.c index 42c229007e..fcf9d759e3 100644 --- a/ext/resindvd/gstmpegdemux.c +++ b/ext/resindvd/gstmpegdemux.c @@ -708,7 +708,11 @@ gst_flups_demux_handle_dvd_event (GstFluPSDemux * demux, GstEvent * event) break; case 0x7: /* FIXME: What range is SDDS? */ - break; + GST_WARNING_OBJECT (demux, + "Unsupported audio stream format in language code event: %d", + stream_format); + temp = NULL; + continue; default: GST_WARNING_OBJECT (demux, "Unknown audio stream format in language code event: %d",