OSDN Git Service

lseek: Correct order of offset arguments
authorAnton Kolesov <Anton.Kolesov@synopsys.com>
Fri, 1 Aug 2014 18:18:47 +0000 (22:18 +0400)
committerBernhard Reutner-Fischer <rep.dot.nop@gmail.com>
Mon, 18 Aug 2014 10:01:36 +0000 (12:01 +0200)
commit361285886aa319a56c803f2fb783457f3e80f564
treeac08937811c6701bb63951c10423146422193257
parent95ec1755ef61629169a79a616ec478988bb6ce7d
lseek: Correct order of offset arguments

There was a runtime error in systems without large file support. Call
fseek(fd, 4096, SEEK_SET) has been failing with EINVAL, though it was
succeeding for offset = 4092. This has been happening because llseek system
call accepts 64-bit value as an offset argument and lseek function has been
ordering 32-bits words that form this offset value, according to the
endianness. However this ordering to match endianness is not required,
because llseek doesn't accept one 64-bit offset argument, it accepts two
32-bit offset argument, then stitches them into one following its
endianness. As a result on little endian system, order of words has been
swapped two time: in libc and in kernel. Thus call to fseek with offset 4096
(0x1000) was doing a system call to llseek with offset 0x1000_0000_0000. I'm
not entirely sure why then offset = 4092 hasn't been failing then.

This patch removes malicious swap of words when calling llseek.

Signed-off-by: Anton Kolesov <Anton.Kolesov@synopsys.com>
Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
libc/sysdeps/linux/common/lseek.c
test/stdio/lseek_no_lfs.c [new file with mode: 0644]