Bug#696278: gst-plugins-bad0.10: Sets FD_CLOEXEC incorrectly with F_SETFL instead of F_SETFD

Guillem Jover guillem at debian.org
Tue Dec 18 22:01:17 UTC 2012


Source: gst-plugins-bad0.10
Version: 0.10.23-7
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-gst-plugins-bad0.10-Set-FD_CLOEXEC-correctly-using-F.patch
Type: text/x-diff
Size: 1601 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-gstreamer-maintainers/attachments/20121218/a28d9344/attachment.patch>


More information about the pkg-gstreamer-maintainers mailing list