[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    Re: [BusyBox] Race condition with tar zxf
From:       Glenn McGrath <bug1 () optushome ! com ! au>
Date:       2001-03-28 9:20:56
[Download RAW message or body]

Robert Kaiser wrote:
> 
> >
> > The gz_open code is mostly taken from tar anyway, i think the forking
> > code for accessing a decompressed stream of data really should go in the
> > gunzip.c rather than in applets prior to calling unzip.
> 
> Agreed. But where do you want put the SIGCHLD handler that is currently in tar ?
> If you think that it is really necessary (which I'm still not convinced of),
> please make sure that it does not just unconditionally exit the parent process
> (see the first of my patches).
> 

The changes ive put into cvs now disable signal handling and allow the
parent process to kill the child when required, so there shouldnt be a
race condition.

I hope this fixes your situation as its a bug that has come up before,
but i wasnt to confident with signals, so wasnt really sure about the
best aproach to the problem.


Glenn



[prev in list] [next in list] [prev in thread] [next in thread]