report at bugs
Aug 15, 2012, 10:46 AM
Post #1 of 3
New submission from Steven Willis:
[issue15676] mmap: add empty file check prior to offset check
There are a number of issues dealing with the offset and length checks in offset, such as issue12556.
I'm running into this issue as well, but with a normal file that happens to be empty. I'm trying to access it with:
mmap.mmap(f.fileno(), length=0, access=mmap.ACCESS_READ)
So the length and offset should be calculated automatically. The man page for mmap says:
"SUSv3 specifies that mmap() should fail if length is 0. However, in kernels before 2.6.12, mmap() succeeded in this case: no mapping was created and the call returned addr. Since kernel 2.6.12, mmap() fails with the error EINVAL for this case."
So alright, mmapping an empty file is now allowed. But, could the check for an empty file be done prior to the check for the offset exceeding the size of the file? It would be much clearer in the cases where an empty (regular or otherwise) file was mmapped if the error message were something like "empty files cannot be mapped" insted of "offset is greater than file size" since I didn't even set the offset.
components: Library (Lib)
title: mmap: add empty file check prior to offset check
versions: Python 2.7, Python 3.1, Python 3.2, Python 3.3, Python 3.4
Python tracker <report [at] bugs>
Python-bugs-list mailing list