Fix CVE-2012-6687 in the bundled fcgi library
This commit is contained in:
parent
c2a21bce21
commit
e8ee1ed51d
84
FCGI-0.78-CVE-2012-6687.patch
Normal file
84
FCGI-0.78-CVE-2012-6687.patch
Normal file
@ -0,0 +1,84 @@
|
||||
Description: fix CVE-2012-6687 in bundled libfcgi
|
||||
Origin: https://bugs.launchpad.net/ubuntu/+source/libfcgi/+bug/933417
|
||||
Bug-Debian: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=815840
|
||||
Forwarded: https://rt.cpan.org/Ticket/Display.html?id=118405
|
||||
|
||||
--- a/os_unix.c
|
||||
+++ b/os_unix.c
|
||||
@@ -36,6 +36,7 @@
|
||||
#include <sys/time.h>
|
||||
#include <sys/un.h>
|
||||
#include <signal.h>
|
||||
+#include <poll.h>
|
||||
|
||||
#ifdef HAVE_NETDB_H
|
||||
#include <netdb.h>
|
||||
@@ -97,6 +98,9 @@
|
||||
static int shutdownPending = FALSE;
|
||||
static int shutdownNow = FALSE;
|
||||
|
||||
+static int libfcgiOsClosePollTimeout = 2000;
|
||||
+static int libfcgiIsAfUnixKeeperPollTimeout = 2000;
|
||||
+
|
||||
void OS_ShutdownPending()
|
||||
{
|
||||
shutdownPending = TRUE;
|
||||
@@ -162,6 +166,16 @@
|
||||
if(libInitialized)
|
||||
return 0;
|
||||
|
||||
+ char *libfcgiOsClosePollTimeoutStr = getenv( "LIBFCGI_OS_CLOSE_POLL_TIMEOUT" );
|
||||
+ if(libfcgiOsClosePollTimeoutStr) {
|
||||
+ libfcgiOsClosePollTimeout = atoi(libfcgiOsClosePollTimeoutStr);
|
||||
+ }
|
||||
+
|
||||
+ char *libfcgiIsAfUnixKeeperPollTimeoutStr = getenv( "LIBFCGI_IS_AF_UNIX_KEEPER_POLL_TIMEOUT" );
|
||||
+ if(libfcgiIsAfUnixKeeperPollTimeoutStr) {
|
||||
+ libfcgiIsAfUnixKeeperPollTimeout = atoi(libfcgiIsAfUnixKeeperPollTimeoutStr);
|
||||
+ }
|
||||
+
|
||||
asyncIoTable = (AioInfo *)malloc(asyncIoTableSize * sizeof(AioInfo));
|
||||
if(asyncIoTable == NULL) {
|
||||
errno = ENOMEM;
|
||||
@@ -751,19 +765,16 @@
|
||||
{
|
||||
if (shutdown(fd, 1) == 0)
|
||||
{
|
||||
- struct timeval tv;
|
||||
- fd_set rfds;
|
||||
+ struct pollfd pfd;
|
||||
int rv;
|
||||
char trash[1024];
|
||||
|
||||
- FD_ZERO(&rfds);
|
||||
+ pfd.fd = fd;
|
||||
+ pfd.events = POLLIN;
|
||||
|
||||
do
|
||||
{
|
||||
- FD_SET(fd, &rfds);
|
||||
- tv.tv_sec = 2;
|
||||
- tv.tv_usec = 0;
|
||||
- rv = select(fd + 1, &rfds, NULL, NULL, &tv);
|
||||
+ rv = poll(&pfd, 1, libfcgiOsClosePollTimeout);
|
||||
}
|
||||
while (rv > 0 && read(fd, trash, sizeof(trash)) > 0);
|
||||
}
|
||||
@@ -1113,13 +1124,11 @@
|
||||
*/
|
||||
static int is_af_unix_keeper(const int fd)
|
||||
{
|
||||
- struct timeval tval = { READABLE_UNIX_FD_DROP_DEAD_TIMEVAL };
|
||||
- fd_set read_fds;
|
||||
-
|
||||
- FD_ZERO(&read_fds);
|
||||
- FD_SET(fd, &read_fds);
|
||||
+ struct pollfd pfd;
|
||||
+ pfd.fd = fd;
|
||||
+ pfd.events = POLLIN;
|
||||
|
||||
- return select(fd + 1, &read_fds, NULL, NULL, &tval) >= 0 && FD_ISSET(fd, &read_fds);
|
||||
+ return poll(&pfd, 1, libfcgiIsAfUnixKeeperPollTimeout) >= 0 && (pfd.revents & POLLIN);
|
||||
}
|
||||
|
||||
/*
|
||||
@ -8,6 +8,9 @@ Release: 11%{?dist}
|
||||
License: OML
|
||||
|
||||
Source0: https://cpan.metacpan.org/authors/id/E/ET/ETHER/FCGI-%{version}.tar.gz
|
||||
# Fix CVE-2012-6687 in the bundled fcgi library, bug #1190294, patch copied
|
||||
# from Debian's libfcgi-perl.
|
||||
Patch0: FCGI-0.78-CVE-2012-6687.patch
|
||||
URL: https://metacpan.org/release/FCGI
|
||||
BuildRequires: coreutils
|
||||
BuildRequires: findutils
|
||||
@ -44,6 +47,7 @@ Provides: bundled(fcgi)
|
||||
|
||||
%prep
|
||||
%setup -q -n FCGI-%{version}
|
||||
%patch0 -p1
|
||||
find . -type f -exec chmod -c -x {} +
|
||||
|
||||
%build
|
||||
@ -67,6 +71,7 @@ make test
|
||||
%changelog
|
||||
* Tue Jan 15 2019 Petr Pisar <ppisar@redhat.com> - 1:0.78-11
|
||||
- Document an fcgi library is bundled (bug #736612)
|
||||
- Fix CVE-2012-6687 in the bundled fcgi library (bug #1190294)
|
||||
|
||||
* Fri Jul 13 2018 Fedora Release Engineering <releng@fedoraproject.org> - 1:0.78-10
|
||||
- Rebuilt for https://fedoraproject.org/wiki/Fedora_29_Mass_Rebuild
|
||||
|
||||
Loading…
Reference in New Issue
Block a user