lash_wrap never exits (fix included)

Bug #730216 reported by danmb
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
lashwrap (Ubuntu)
Fix Released
Undecided
Unassigned
Natty
Won't Fix
Undecided
Unassigned
Oneiric
Fix Released
Undecided
Unassigned

Bug Description

Binary package hint: lashwrap

$ lash_wrap -- sleep 3600
$ killall sleep
$ sleep 5; pidof lash_wrap
10234

lash_wrap doesn't exit when the wrapped application terminates, because it improperly tries to ignore SIGCHLD. From the waitpid manpage:

Note that even though the default disposition of SIGCHLD is "ignore", explicitly setting the disposition to SIG_IGN results in different treatment of zombie process children.

The fix is simple -- remove the line that says

signal(SIGCHLD, SIG_IGN)

It's unnecessary and it harmful.

ProblemType: Bug
DistroRelease: Ubuntu 10.04
Package: lashwrap 1.0.2-0ubuntu2
ProcVersionSignature: Ubuntu 2.6.31-11.154-rt
Uname: Linux 2.6.31-11-rt i686
NonfreeKernelModules: nvidia
Architecture: i386
Date: Sun Mar 6 19:21:58 2011
ProcEnviron:
 LC_COLLATE=C
 PATH=(custom, user)
 LANG=en_US.UTF-8
 SHELL=/bin/bash
SourcePackage: lashwrap

Related branches

Revision history for this message
danmb (danmbox) wrote :
danmb (danmbox)
Changed in lashwrap (Ubuntu):
status: New → Incomplete
danmb (danmbox)
Changed in lashwrap (Ubuntu):
status: Incomplete → New
Revision history for this message
danmb (danmbox) wrote :

Patch form for this fix, even though it's trivial...

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

This bug was fixed in the package lashwrap - 1.0.2-0ubuntu7

---------------
lashwrap (1.0.2-0ubuntu7) oneiric; urgency=low

  * Apply patch from Dan Muresan resolving lash_wrap not exiting due
    to an unnecessary signal handler being installed (LP: #730216).
 -- Daniel T Chen <email address hidden> Fri, 22 Jul 2011 11:36:27 -0400

Changed in lashwrap (Ubuntu Oneiric):
status: New → Fix Released
Revision history for this message
Rolf Leggewie (r0lf) wrote :

natty has seen the end of its life and is no longer receiving any updates. Marking the natty task for this ticket as "Won't Fix".

Changed in lashwrap (Ubuntu Natty):
status: New → Won't Fix
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.