Skip to content

Commit 5dfc547

Browse files
committed
avutil/file: use av_err2str to simplify code
No need to explicitly specify the buffer here as it is only ever passed to av_log, so av_err2str can be used.
1 parent 074bf9f commit 5dfc547

File tree

1 file changed

+3
-7
lines changed

1 file changed

+3
-7
lines changed

libavutil/file.c

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -60,21 +60,18 @@ int av_file_map(const char *filename, uint8_t **bufptr, size_t *size,
6060
struct stat st;
6161
av_unused void *ptr;
6262
off_t off_size;
63-
char errbuf[128];
6463
*bufptr = NULL;
6564
*size = 0;
6665

6766
if (fd < 0) {
6867
err = AVERROR(errno);
69-
av_strerror(err, errbuf, sizeof(errbuf));
70-
av_log(&file_log_ctx, AV_LOG_ERROR, "Cannot read file '%s': %s\n", filename, errbuf);
68+
av_log(&file_log_ctx, AV_LOG_ERROR, "Cannot read file '%s': %s\n", filename, av_err2str(err));
7169
return err;
7270
}
7371

7472
if (fstat(fd, &st) < 0) {
7573
err = AVERROR(errno);
76-
av_strerror(err, errbuf, sizeof(errbuf));
77-
av_log(&file_log_ctx, AV_LOG_ERROR, "Error occurred in fstat(): %s\n", errbuf);
74+
av_log(&file_log_ctx, AV_LOG_ERROR, "Error occurred in fstat(): %s\n", av_err2str(err));
7875
close(fd);
7976
return err;
8077
}
@@ -97,8 +94,7 @@ int av_file_map(const char *filename, uint8_t **bufptr, size_t *size,
9794
ptr = mmap(NULL, *size, PROT_READ|PROT_WRITE, MAP_PRIVATE, fd, 0);
9895
if (ptr == MAP_FAILED) {
9996
err = AVERROR(errno);
100-
av_strerror(err, errbuf, sizeof(errbuf));
101-
av_log(&file_log_ctx, AV_LOG_ERROR, "Error occurred in mmap(): %s\n", errbuf);
97+
av_log(&file_log_ctx, AV_LOG_ERROR, "Error occurred in mmap(): %s\n", av_err2str(err));
10298
close(fd);
10399
*size = 0;
104100
return err;

0 commit comments

Comments
 (0)