Bug#696295: webkit: Sets FD_CLOEXEC incorrectly with F_SETFL instead of F_SETFD

[Guillem Jover]

Source: webkit
Version: 1.9.2-1
Severity: important
Tags: patch
User: debian-bsd at lists.debian.org
Usertags: fcntl-fd-cloexec

Hi!

This package contains code that tries to set the FD_CLOEXEC flag for a
file descriptor, but it does using F_SETFL instead of F_SETFD.

Using that value on F_SETFL is just wrong, and might make the call fail
on some systems, as it's requesting to set an undetermined flag. For
example on GNU/* FD_CLOEXEC has value 1, which matches with O_WRONLY.

This might cause the code to at least leak file descriptors, and at
worst to terminate execution.

Attached a patch fixing this.

Thanks,
Guillem
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-webkit-Set-FD_CLOEXEC-correctly-using-F_SETFD-not-F_.patch
Type: text/x-diff
Size: 1531 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-webkit-maintainers/attachments/20121219/e435d657/attachment.patch>