Fix socket leaks in the BrowsePoll code (bug #1021512).

Resolves: rhbz#1021512
This commit is contained in:
Tim Waugh 2013-10-21 14:04:47 +01:00
parent a267035fb9
commit 625d209b25
2 changed files with 30 additions and 1 deletions

View File

@ -0,0 +1,22 @@
diff -up cups-filters-1.0.40/utils/cups-browsed.c.socket-leak cups-filters-1.0.40/utils/cups-browsed.c
--- cups-filters-1.0.40/utils/cups-browsed.c.socket-leak 2013-10-02 08:53:26.000000000 +0100
+++ cups-filters-1.0.40/utils/cups-browsed.c 2013-10-21 14:03:11.907471644 +0100
@@ -1677,6 +1677,8 @@ browse_poll_cancel_subscription (browsep
if (response)
ippDelete(response);
+
+ httpClose (conn);
}
static gboolean
@@ -1787,6 +1789,9 @@ browse_poll (gpointer data)
fail:
+ if (conn)
+ httpClose (conn);
+
/* Call a new timeout handler so that we run again */
g_timeout_add_seconds (BrowseInterval, browse_poll, data);

View File

@ -4,7 +4,7 @@
Summary: OpenPrinting CUPS filters and backends
Name: cups-filters
Version: 1.0.40
Release: 3%{?dist}
Release: 4%{?dist}
# For a breakdown of the licensing, see COPYING file
# GPLv2: filters: commandto*, imagetoraster, pdftops, rasterto*,
@ -22,6 +22,7 @@ Url: http://www.linuxfoundation.org/collaborate/workgroups/openprinting/cups
Source0: http://www.openprinting.org/download/cups-filters/cups-filters-%{version}.tar.xz
Patch1: cups-filters-pdf-landscape.patch
Patch2: cups-filters-socket-leak.patch
Requires: cups-filters-libs%{?_isa} = %{version}-%{release}
@ -107,6 +108,9 @@ This is the development package for OpenPrinting CUPS filters and backends.
# Fix PDF landscape printing (bug #768811).
%patch1 -p1 -b .pdf-landscape
# Fix socket leaks in the BrowsePoll code (bug #1021512).
%patch2 -p1 -b .socket-leak
%build
# work-around Rpath
./autogen.sh
@ -220,6 +224,9 @@ fi
%{_libdir}/libfontembed.so
%changelog
* Mon Oct 21 2013 Tim Waugh <twaugh@redhat.com> - 1.0.40-4
- Fix socket leaks in the BrowsePoll code (bug #1021512).
* Wed Oct 16 2013 Tim Waugh <twaugh@redhat.com> - 1.0.40-3
- Ship the gstoraster MIME conversion rule now we provide that filter
(bug #1019261).