Re: [PATCH] daemonize & change user fixes

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.

-- 
 &Kalle Marjola ::: Development ::: Helsinki ::: Enpocket