[FFmpeg-devel] [PATCH] avutil/hwcontext_drm: Check ioctl in drm_map_frame() for failure
Michael Niedermayer
michael at niedermayer.cc
Mon May 20 05:19:44 EEST 2024
Fixes: CID1583742 Unchecked return value
Sponsored-by: Sovereign Tech Fund
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
---
libavutil/hwcontext_drm.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/libavutil/hwcontext_drm.c b/libavutil/hwcontext_drm.c
index 0847db09a08..e080c0597b8 100644
--- a/libavutil/hwcontext_drm.c
+++ b/libavutil/hwcontext_drm.c
@@ -166,7 +166,10 @@ static int drm_map_frame(AVHWFramesContext *hwfc,
#if HAVE_LINUX_DMA_BUF_H
/* We're not checking for errors here because the kernel may not
* support the ioctl, in which case its okay to carry on */
- ioctl(desc->objects[i].fd, DMA_BUF_IOCTL_SYNC, &sync_start);
+ if (ioctl(desc->objects[i].fd, DMA_BUF_IOCTL_SYNC, &sync_start) == -1) {
+ err = AVERROR(errno);
+ goto fail;
+ }
#endif
}
map->nb_regions = i;
--
2.45.1
More information about the ffmpeg-devel
mailing list