Re: Race condition in ldif2ldbm (ITS#140)

[dave@chello.com]

sasgwi@wnt.sas.com wrote:
> 
> I think this would do the same job:
> 
> diff -r1.25 ldif2ldbm.c
> 60a61
> >     int     cmdkids = 1;
> 90c91
> <                       maxkids = atoi( optarg );
> ---
> >                       cmdkids = atoi( optarg );
> 171a173,174
> >
> >     maxkids = cmdkids;
> 

(Sorry for taking so long to reply to this: I've been looking at other
things
lately.)

The above doesn't work. In fact, it makes absolutely no difference to
the operation of ldif2ldbm. Think about what happens: id2entry gets
forked,
then cmdkids gets written with the same value it would have on startup,
so ldif2ldbm carries on exactly as before.

Please back out the above 'fix' from the development branch, because it
wasn't tested, and doesn't work.

	- Dave