decklink: Update enum value bounds check in gst_decklink_get_mode()
authorSebastian Dröge <sebastian@centricular.com>
Fri, 30 Oct 2020 08:02:32 +0000 (10:02 +0200)
committerSebastian Dröge <sebastian@centricular.com>
Fri, 30 Oct 2020 08:02:32 +0000 (10:02 +0200)
The widescreen modes moved after GST_DECKLINK_MODE_2160p60 and using
them now would cause an assertion. This is a regression from
309f6187fef890c7ffa49305f38e89beac3b1423.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/-/merge_requests/1737>

sys/decklink/gstdecklink.cpp

index 5637b10..3599126 100644 (file)
@@ -425,7 +425,7 @@ static const struct
 const GstDecklinkMode *
 gst_decklink_get_mode (GstDecklinkModeEnum e)
 {
-  if (e < GST_DECKLINK_MODE_AUTO || e > GST_DECKLINK_MODE_2160p60)
+  if (e < GST_DECKLINK_MODE_AUTO || e > GST_DECKLINK_MODE_PAL_P_WIDESCREEN)
     return NULL;
   return &modes[e];
 }