Applied.
On Jan 5, 2008 3:28 PM, James Youngman <[EMAIL PROTECTED]> wrote: > 2008-01-05 James Youngman <[EMAIL PROTECTED]> > > * xargs/xargs.c: (main): Standardise on "Warning" instead of > "warning" in messages. > > * xargs/xargs.c: (add_proc): Use x2nrealloc to extend the pids > array, rather than doubling the size of the buffer (since the old > aproach was vulnerable to overflow). > > Reap all available child processes before every fork. This fixes > Savannah bug #21960. > * xargs/xargs.c: (proc_max): since this is a non-negative > quantity, make it unsigned. > (procs_executing): Likewise. > (pids_alloc): Likewise (using size_t). > (procs_executed): In order to prevent possible overflow, make this > a boolean, not a count. We only cared if the previous counter was > zero or not, anwyay. > (add_proc): Set procs_executed to true rather than incrementing it. > (wait_for_proc): When called, always reap all available children. > Add an extra argument which is the minimum number of children we > must reap before returning. > (wait_for_proc_all): Pass the new extra argument. > (xargs_do_exec): Call wait_for_proc() to reap all available > children before forking a new child. Modify other calls to > wait_for_proc to pass the new extra argument. > (NEWS): Mention this change. _______________________________________________ Findutils-patches mailing list Findutils-patches@gnu.org http://lists.gnu.org/mailman/listinfo/findutils-patches