On 11/30/24 9:23 AM, Niklas Haas wrote:
From: Niklas Haas <g...@haasn.dev>
---
libavutil/csp.c | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/libavutil/csp.c b/libavutil/csp.c
index 7ef822c60b..3dd7bc2562 100644
--- a/libavutil/csp.c
+++ b/libavutil/csp.c
@@ -290,11 +290,7 @@ static const av_csp_trc_function trc_funcs[AVCOL_TRC_NB] =
{
av_csp_trc_function av_csp_trc_func_from_id(enum AVColorTransferCharacteristic trc)
{
- av_csp_trc_function func;
if (trc >= AVCOL_TRC_NB)
return NULL;
- func = trc_funcs[trc];
- if (!func)
- return NULL;
- return func;
+ return trc_funcs[trc];
}
This is future-proof in case there's gaps in H.273, which there are in
some cases.
Also it prevents a segfault crash if someone passes an enum that hasn't
been added to FFmpeg yet, but has been added to H.273.
- Leo Izen
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".