socket reconnection thread is not started

Bug #785134 reported by Peter Soetens
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
log4cxx (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

There is a tiny bug in the reconnection logic of socketappender such that the thread that needs to do the reconnection is never started.

I have created a trivial patch that fixes this. Tested on maverick, will have same effect on all other releases too.

Tags: patch

Related branches

Revision history for this message
Peter Soetens (peter-soetens) wrote :
Revision history for this message
Peter Soetens (peter-soetens) wrote :

The bug seemed more involved than at first sight. The proposed patch only fixes the case where the thread is run once by the socket appender, but any next attempts because the connection was dropped *again* will still not happen.

A second patch that replaces the first one fixes all attempts of reconnection.

I also ran the unit tests to be sure it didn't break anything. All went fine.

tags: added: patch
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package log4cxx - 0.10.0-1.2ubuntu1

---------------
log4cxx (0.10.0-1.2ubuntu1) oneiric; urgency=low

  * Merge from debian/unstable, new changes:
    - debian/patches/130-bugfix-LOGCXX-322.dpatch:
      patch from Stephan Sürke to fix a potential segfault on exit
      (Closes: #578649)
    - debian/patches/140-fix-socket-thread-not-starting.dpatch:
      patch from Peter Soertens to fix an issue with the socket
      reconnection thread not starting (LP: #785134)

log4cxx (0.10.0-1.2) unstable; urgency=low

  * Non-maintainer upload.
  * Don't ship .la files (Closes: #522197, #622538).
 -- Andreas Moog <email address hidden> Sun, 03 Jul 2011 10:22:17 +0000

Changed in log4cxx (Ubuntu):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.