diff mbox series

[FFmpeg-devel] fftools/ffmpeg_filter: use a bprint buffer for the crop filter arguments

Message ID 20240710175518.15937-1-jamrial@gmail.com
State New
Headers show
Series [FFmpeg-devel] fftools/ffmpeg_filter: use a bprint buffer for the crop filter arguments | expand

Checks

Context Check Description
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

James Almer July 10, 2024, 5:55 p.m. UTC 
Should ensure no truncation ocurrs.

The input sample aspect ratio applies to the pre-cropping dimensions, so update it.

Signed-off-by: James Almer <jamrial@gmail.com>
---
 fftools/ffmpeg_filter.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
diff mbox series

Patch

diff --git a/fftools/ffmpeg_filter.c b/fftools/ffmpeg_filter.c
index 097bd2ed48..bb88c6711a 100644
--- a/fftools/ffmpeg_filter.c
+++ b/fftools/ffmpeg_filter.c
@@ -1702,12 +1702,12 @@  static int configure_input_video_filter(FilterGraph *fg, AVFilterGraph *graph,
     av_assert0(desc);
 
     if ((ifp->opts.flags & IFILTER_FLAG_CROP)) {
-        char crop_buf[64];
-        snprintf(crop_buf, sizeof(crop_buf), "w=iw-%d-%d:h=ih-%d-%d:x=%d:y=%d",
+        av_bprint_clear(&args);
+        av_bprintf(&args, "w=iw-%d-%d:h=ih-%d-%d:x=%d:y=%d",
                  ifp->opts.crop_left, ifp->opts.crop_right,
                  ifp->opts.crop_top, ifp->opts.crop_bottom,
                  ifp->opts.crop_left, ifp->opts.crop_top);
-        ret = insert_filter(&last_filter, &pad_idx, "crop", crop_buf);
+        ret = insert_filter(&last_filter, &pad_idx, "crop", args.str);
         if (ret < 0)
             return ret;
     }