commit 047dc617e617a00662b2f03fa8f1e7d00b0e013e
author Nicolas Dufresne <nicolas.dufresne@collabora.com> Thu Jun 28 19:08:35 2018 -0400
committer Nicolas Dufresne <nicolas.dufresne@collabora.com> Fri Jun 29 10:39:34 2018 -0400
tree 0d38dc78ff59b0abe2fb5fbf850373853bcceb98
parent e8f8cd2ce02442f359415013e853afc098c75d22

v4l2videoenc: Only renegotiate with upstream

When the decoder get linked further, it will receive a renegotiation
event from downstream. This case is not supported and should be ignored.

This fixes issues when this encoder is used inside an GstRtspServer
pipeline.

https://bugzilla.gnome.org/show_bug.cgi?id=796525

sys/v4l2/gstv4l2videoenc.c

diff --git a/sys/v4l2/gstv4l2videoenc.c b/sys/v4l2/gstv4l2videoenc.c
index 5b6b810..7e88923 100644
--- a/sys/v4l2/gstv4l2videoenc.c
+++ b/sys/v4l2/gstv4l2videoenc.c
@@ -521,6 +521,10 @@
   GST_DEBUG_OBJECT (self, "Negotiating %s profile and level.",
       klass->codec_name);
 
+  /* Only renegotiate on upstream changes */
+  if (self->input_state)
+    return TRUE;
+
   allowed_caps = gst_pad_get_allowed_caps (GST_VIDEO_ENCODER_SRC_PAD (encoder));
 
   if (allowed_caps) {