[FFmpeg-devel] [PATCH 1/2] avutil/file: do not request PROT_WRITE in av_file_map

Marton Balint cus at passwd.hu
Thu Sep 6 21:58:36 EEST 2018


The documentation states that av_file_map can only be used for reading and for
WIN32 we are already requesting a read only mapping.

Signed-off-by: Marton Balint <cus at passwd.hu>
---
 libavutil/file.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavutil/file.c b/libavutil/file.c
index 24a86c3f35..2153e51379 100644
--- a/libavutil/file.c
+++ b/libavutil/file.c
@@ -86,7 +86,7 @@ int av_file_map(const char *filename, uint8_t **bufptr, size_t *size,
     *size = off_size;
 
 #if HAVE_MMAP
-    ptr = mmap(NULL, *size, PROT_READ|PROT_WRITE, MAP_PRIVATE, fd, 0);
+    ptr = mmap(NULL, *size, PROT_READ, MAP_PRIVATE, fd, 0);
     if (ptr == MAP_FAILED) {
         err = AVERROR(errno);
         av_strerror(err, errbuf, sizeof(errbuf));
-- 
2.16.4



More information about the ffmpeg-devel mailing list