]> git.baikalelectronics.ru Git - kernel.git/commit
fs/pipe.c: preserve alloc_file() error code
authorEric Biggers <ebiggers3@gmail.com>
Sat, 17 Oct 2015 21:26:08 +0000 (16:26 -0500)
committerAl Viro <viro@zeniv.linux.org.uk>
Wed, 11 Nov 2015 07:18:23 +0000 (02:18 -0500)
commit0bb2cb6f3c23ea48628f4f58901a29867b0f95c4
treecbc932082f5dba8bd38b01d444ecf256a6e1fbc6
parenta9f586b9307b052e8607c403f65e81ba2b6c6005
fs/pipe.c: preserve alloc_file() error code

If sys_pipe() was unable to allocate a 'struct file', it always failed
with ENFILE, which means "The number of simultaneously open files in the
system would exceed a system-imposed limit." However, alloc_file()
actually returns an ERR_PTR value and might fail with other error codes.
Currently, in addition to ENFILE, it can fail with ENOMEM, potentially
when there are few open files in the system.  Update sys_pipe() to
preserve this error code.

In a prior submission of a similar patch (1) some concern was raised
about introducing a new error code for sys_pipe().  However, for most
system calls, programs cannot assume that new error codes will never be
introduced.  In addition, ENOMEM was, in fact, already a possible error
code for sys_pipe(), in the case where the file descriptor table could
not be expanded due to insufficient memory.

(1) http://comments.gmane.org/gmane.linux.kernel/1357942

Signed-off-by: Eric Biggers <ebiggers3@gmail.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/pipe.c