Hi,
Kalle, do you still have objections for this patch or is it ok to commit as
is?
Kalle Marjola wrote:
> On Tue, 2004-11-02 at 02:48, Alexander Malysh wrote:
>> Hi,
>>
>> attached patch fixes daemonize mode (make sure stdin/stdout/sdterr are
>> opened and do chdir("/")) and change user code (set supplementary group
>> id's and don't destroy passwd struct).
>>
> +1 for all except that '/' -thing:
> -----
> /* XXX chdir breaks restart of boxes when
> started w/o a full path to binary */
> - /* chdir("/"); */
> + chdir("/");
> -----
> Seems like there was reason not to do it. I do not say that
> this isn't right thing to do, but then if parachute is used,
> then Kannel should make sure that the path is full, not
> relative - can it like test, in daemon mode, that it can find
> itself before starting up?
>
>> This patch should go into 1.4.0 stable.
>
> Yes except notes above.
>
--
Thanks,
Alex
