[FFmpeg-devel] [PATCH 7/7] tools/aviocat: use av_err2str

Marvin Scholz epirat07 at gmail.com
Wed Apr 2 02:43:36 EEST 2025


There is no need to explicitly specify the buffer, as it
is only ever passed to fprintf, so av_err2str can be used.
---
 tools/aviocat.c | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/tools/aviocat.c b/tools/aviocat.c
index 816ab700ef..7f8a247f1c 100644
--- a/tools/aviocat.c
+++ b/tools/aviocat.c
@@ -37,7 +37,6 @@ int main(int argc, char **argv)
     const char *input_url = NULL, *output_url = NULL;
     int64_t stream_pos = 0;
     int64_t start_time;
-    char errbuf[50];
     AVIOContext *input, *output;
     AVDictionary *in_opts = NULL;
     AVDictionary *out_opts = NULL;
@@ -80,8 +79,7 @@ int main(int argc, char **argv)
 
     ret = avio_open2(&input, input_url, AVIO_FLAG_READ, NULL, &in_opts);
     if (ret) {
-        av_strerror(ret, errbuf, sizeof(errbuf));
-        fprintf(stderr, "Unable to open %s: %s\n", input_url, errbuf);
+        fprintf(stderr, "Unable to open %s: %s\n", input_url, av_err2str(ret));
         return 1;
     }
     if (verbose) {
@@ -95,16 +93,14 @@ int main(int argc, char **argv)
     if (duration && !bps) {
         int64_t size = avio_size(input);
         if (size < 0) {
-            av_strerror(ret, errbuf, sizeof(errbuf));
-            fprintf(stderr, "Unable to get size of %s: %s\n", input_url, errbuf);
+            fprintf(stderr, "Unable to get size of %s: %s\n", input_url, av_err2str(ret));
             goto fail;
         }
         bps = size / duration;
     }
     ret = avio_open2(&output, output_url, AVIO_FLAG_WRITE, NULL, &out_opts);
     if (ret) {
-        av_strerror(ret, errbuf, sizeof(errbuf));
-        fprintf(stderr, "Unable to open %s: %s\n", output_url, errbuf);
+        fprintf(stderr, "Unable to open %s: %s\n", output_url, av_err2str(ret));
         goto fail;
     }
 
@@ -117,8 +113,7 @@ int main(int argc, char **argv)
             break;
         avio_write(output, buf, n);
         if (output->error) {
-            av_strerror(output->error, errbuf, sizeof(errbuf));
-            fprintf(stderr, "Unable to write %s: %s\n", output_url, errbuf);
+            fprintf(stderr, "Unable to write %s: %s\n", output_url, av_err2str(output->error));
             break;
         }
         stream_pos += n;
-- 
2.39.5 (Apple Git-154)



More information about the ffmpeg-devel mailing list