On Tue, May 16, 2017 at 04:48:13PM +0200, Victor Toso wrote: > From: Victor Toso <m...@victortoso.com> > > Inserting this check in channel-display-gst.c as the GStreamer decoder > is the only one handling all the different video formats supported by > spice-protocol. > > If a unsupported/bad codec type value was sent, spice-gtk will fail to > create the decoder and any messages related to this stream-id will be > ignored. > > Signed-off-by: Victor Toso <victort...@redhat.com> > Signed-off-by: Victor Toso <m...@victortoso.com>
Annoying double signed-off.. I'll remove before pushing... (if acked)
> ---
> src/channel-display-gst.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/src/channel-display-gst.c b/src/channel-display-gst.c
> index 9b79403..d3e83e3 100644
> --- a/src/channel-display-gst.c
> +++ b/src/channel-display-gst.c
> @@ -512,6 +512,8 @@ VideoDecoder* create_gstreamer_decoder(int codec_type,
> display_stream *stream)
> {
> SpiceGstDecoder *decoder = NULL;
>
> + g_return_val_if_fail(VALID_VIDEO_CODEC_TYPE(codec_type), NULL);
> +
> if (gstvideo_init()) {
> decoder = spice_new0(SpiceGstDecoder, 1);
> decoder->base.destroy = spice_gst_decoder_destroy;
> --
> 2.13.0
>
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/spice-devel
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/spice-devel
