Your message dated Tue, 10 Mar 2009 12:17:03 +0000
with message-id <[email protected]>
and subject line Bug#516532: fixed in consolekit 0.3.0-1
has caused the Debian Bug report #516532,
regarding consolekit: leaks file descriptors when reading run-session.d
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
516532: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=516532
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: consolekit
Version: 0.2.10-5
Tags: patch

After a machine with consolekit installed has been running for a while,
I see messages in syslog a la:

Feb 21 20:12:38 zaphod.acm.uiuc.edu console-kit-daemon[19801]: WARNING: Unable 
to open directory /etc/ConsoleKit/run-session.d: Error opening directory 
'/etc/ConsoleKit/run-session.d': Too many open files
Feb 21 20:12:38 zaphod.acm.uiuc.edu console-kit-daemon[19801]: WARNING: Unable 
to open directory /usr/lib/ConsoleKit/run-session.d: Error opening directory 
'/usr/lib/ConsoleKit/run-session.d': Too many open files
Feb 21 20:12:38 zaphod.acm.uiuc.edu console-kit-daemon[19801]: WARNING: Cannot 
create file /var/run/ConsoleKit/database~: Too many open files
Feb 21 20:12:38 zaphod.acm.uiuc.edu console-kit-daemon[19801]: WARNING: Cannot 
unlink /var/run/ConsoleKit/database: No such file or directory
Feb 21 20:12:38 zaphod.acm.uiuc.edu console-kit-daemon[19801]: WARNING: Unable 
to open directory /etc/ConsoleKit/run-session.d: Error opening directory 
'/etc/ConsoleKit/run-session.d': Too many open files
Feb 21 20:12:38 zaphod.acm.uiuc.edu console-kit-daemon[19801]: WARNING: Unable 
to open directory /usr/lib/ConsoleKit/run-session.d: Error opening directory 
'/usr/lib/ConsoleKit/run-session.d': Too many open files

And lsof shows that console-kit-daemon has a ton of FDs open for the two
run-session.d directories mentioned above. consolekit opens these
directories, but never closes the fd. The attached patch should fix it.
I'm not able to test it myself right now, since #516253 is preventing me
from building the package (easily).

-- 
Andrew Deason
[email protected]
diff -Nru consolekit-0.2.10/debian/patches/06-close-runsession.patch consolekit-0.2.10.new/debian/patches/06-close-runsession.patch
--- consolekit-0.2.10/debian/patches/06-close-runsession.patch	1969-12-31 18:00:00.000000000 -0600
+++ consolekit-0.2.10.new/debian/patches/06-close-runsession.patch	2009-02-21 23:08:29.013066818 -0600
@@ -0,0 +1,11 @@
+Closes the file handle for /etc/ConsoleKit/run-session.d and
+/usr/lib/ConsoleKit/run-session.d, avoiding a file descriptor leak.
+
+--- ConsoleKit-0.2.10.orig/src/ck-run-programs.c	2008-01-23 08:30:44.000000000 -0600
++++ ConsoleKit-0.2.10/src/ck-run-programs.c	2009-02-21 23:05:34.285066017 -0600
+@@ -227,4 +227,5 @@
+         }
+ out:
+         g_strfreev (env_for_child);
++        g_dir_close (dir);
+ }
diff -Nru consolekit-0.2.10/debian/patches/series consolekit-0.2.10.new/debian/patches/series
--- consolekit-0.2.10/debian/patches/series	2009-02-21 23:38:47.000000000 -0600
+++ consolekit-0.2.10.new/debian/patches/series	2009-02-21 23:08:40.118340806 -0600
@@ -3,3 +3,4 @@
 03-cleanup_console_tags.patch
 04-defer_daemonizing.patch
 05-dbus_policy.patch
+06-close-runsession.patch

--- End Message ---
--- Begin Message ---
Source: consolekit
Source-Version: 0.3.0-1

We believe that the bug you reported is fixed in the latest version of
consolekit, which is due to be installed in the Debian FTP archive:

consolekit_0.3.0-1.diff.gz
  to pool/main/c/consolekit/consolekit_0.3.0-1.diff.gz
consolekit_0.3.0-1.dsc
  to pool/main/c/consolekit/consolekit_0.3.0-1.dsc
consolekit_0.3.0-1_i386.deb
  to pool/main/c/consolekit/consolekit_0.3.0-1_i386.deb
consolekit_0.3.0.orig.tar.gz
  to pool/main/c/consolekit/consolekit_0.3.0.orig.tar.gz
libck-connector-dev_0.3.0-1_i386.deb
  to pool/main/c/consolekit/libck-connector-dev_0.3.0-1_i386.deb
libck-connector0_0.3.0-1_i386.deb
  to pool/main/c/consolekit/libck-connector0_0.3.0-1_i386.deb
libpam-ck-connector_0.3.0-1_i386.deb
  to pool/main/c/consolekit/libpam-ck-connector_0.3.0-1_i386.deb



A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to [email protected],
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Michael Biebl <[email protected]> (supplier of updated consolekit package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing [email protected])


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.8
Date: Tue, 10 Mar 2009 12:48:43 +0100
Source: consolekit
Binary: consolekit libck-connector0 libck-connector-dev libpam-ck-connector
Architecture: source i386
Version: 0.3.0-1
Distribution: unstable
Urgency: low
Maintainer: Utopia Maintenance Team 
<[email protected]>
Changed-By: Michael Biebl <[email protected]>
Description: 
 consolekit - framework for defining and tracking users, sessions and seats
 libck-connector-dev - ConsoleKit development files
 libck-connector0 - ConsoleKit libraries
 libpam-ck-connector - ConsoleKit PAM module
Closes: 489032 511270 516253 516532
Changes: 
 consolekit (0.3.0-1) unstable; urgency=low
 .
   * New upstream release.
   * debian/patches/01-dbus_policy.patch
     - Removed, the access to the restart and stop actions is now properly
       restricted by checks in the source code.
   * debian/patches/02-exit_with_dbus.patch
     - Removed, merged upstream.
   * debian/patches/05-dbus_policy.patch
     - Updated by pulling latest patch from upstream Git.
   * debian/patches/06-xmlto_skip_validation.patch
     - Skip XML validation when generating the documentation. (Closes: #516253)
       A proper fix will be, to make ConsoleKit.xml actually valid XML.
   * debian/consolekit.docs
     - Update path to spec html file.
   * Merge patches from Ubuntu
     - debian/patches/04-consolekit_park.patch (Closes: #489032)
     - debian/patches/05-pam-add-nox11.patch
     - debian/patches/08-fix-double-close.patch
     - debian/patches/09-directory-handle-leak.patch (Closes: #516532)
     - debian/patches/10-file_monitor_remove_watch_crash.patch
     - debian/patches/11-unref-dbus-proxy.patch
     - debian/patches/12-allow-get-sessions.patch
   * debian/control
     - Wrap Build-Depends.
     - Bump Build-Depends on debhelper to (>= 7).
   * debian/compat
     - Bump debhelper compat level to 7.
   * Fix FTBFS on GNU/kFreeBSD. (Closes: #511270)
     - debian/patches/80-gnukfreebsd_support_configure.ac.patch
       debian/patches/81-gnukfreebsd_support_pseudo_run_autoconf.patch
       + Add kfreebsd target to configure(.ac) script
     - debian/control
       + Add libkvm-dev to Build-Depends on kfreebsd ports.
     Patch by Luca Favatella, thanks.
   * debian/libpam-ck-connector.{pam-auth-update,install,postinst,prerm},
     debian/rules: enable pam_ck_connector by default using the new
     pam-auth-update support.
   * debian/control: depend on libpam-runtime (>= 1.0.1-6) for the above.
Checksums-Sha1: 
 d3d7324d0c01786814a4743981888f89aa8428de 1523 consolekit_0.3.0-1.dsc
 e78229c04ce3cf5e2c93f74e4d3dcaca3c15e07b 544749 consolekit_0.3.0.orig.tar.gz
 6d440a42c896abe7f17988dbc37adc348e17d8ef 13648 consolekit_0.3.0-1.diff.gz
 65ff688eaf0e794b8fa8a01476b0260c3506d240 137036 consolekit_0.3.0-1_i386.deb
 e84004605a6e90cd1037a8c36b7a108cdbf71b1a 46802 
libck-connector0_0.3.0-1_i386.deb
 5fd30d54a6d7f80fd692c3609565113ce743cb73 55636 
libck-connector-dev_0.3.0-1_i386.deb
 4f5965d0c48b5d21174f6da4af0d2e19557eef4b 47392 
libpam-ck-connector_0.3.0-1_i386.deb
Checksums-Sha256: 
 7b3fffbadb6021e152ef1f53938c607101bf66c46e80d777b88fc58ba82d37d6 1523 
consolekit_0.3.0-1.dsc
 0070be2c9915391f3d6fb1b44b322e7238c8e76f78d26a043086ee238905cbf4 544749 
consolekit_0.3.0.orig.tar.gz
 009e102043826a74f2a1569c60fd8468b889bb26f82c9147fec475945c183da8 13648 
consolekit_0.3.0-1.diff.gz
 950e1d4e82e59b9b61bc95be5cdbfda33debf39168a81fe6d97a831e48f5a0c3 137036 
consolekit_0.3.0-1_i386.deb
 d0a7e367b18dc09101292c0b8e42614ebc828d34f11e18c9ecebe744e5c9f330 46802 
libck-connector0_0.3.0-1_i386.deb
 9001432dfb42a873de5bd19b45d4f7c4679a7499cf4e995a71e2a78ef6e3bb58 55636 
libck-connector-dev_0.3.0-1_i386.deb
 e2d83194476642260ad67a05bf15e61e4af0ff9d460f04076516611ddc44038f 47392 
libpam-ck-connector_0.3.0-1_i386.deb
Files: 
 aeb54781801963860cfe64c12d41f458 1523 admin optional consolekit_0.3.0-1.dsc
 7bfd109f67be0c975f686200085e1c18 544749 admin optional 
consolekit_0.3.0.orig.tar.gz
 147897aedc0d4703ec9055f39548b0a5 13648 admin optional 
consolekit_0.3.0-1.diff.gz
 0148147cefd46f9b6e5fd9a8e3e74401 137036 admin optional 
consolekit_0.3.0-1_i386.deb
 1f2a5948745207ec6f1a0be2d56da632 46802 libs optional 
libck-connector0_0.3.0-1_i386.deb
 11b37f9b690a87c521bad828ca85503c 55636 libdevel optional 
libck-connector-dev_0.3.0-1_i386.deb
 0c22afd3d1be41a8b8b760b108bbd6bc 47392 admin optional 
libpam-ck-connector_0.3.0-1_i386.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)

iEYEARECAAYFAkm2VGAACgkQh7PER70FhVTerwCfYdNQrzYaoB4jWJbSozD6J+gZ
wt0AoIV5HUqXJoOkFhIeRVE4fop/lhaC
=UfF+
-----END PGP SIGNATURE-----



--- End Message ---
_______________________________________________
Pkg-utopia-maintainers mailing list
[email protected]
http://lists.alioth.debian.org/mailman/listinfo/pkg-utopia-maintainers

Reply via email to