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

List:       busybox
Subject:    Re: [bug] [patch] fuser.c:
From:       Denys Vlasenko <vda.linux () googlemail ! com>
Date:       2008-05-28 14:56:06
Message-ID: 200805281656.06415.vda.linux () googlemail ! com
[Download RAW message or body]

On Wednesday 28 May 2008 15:31, Cristian Ionescu-Idbohrn wrote:
> Are we absolutely sure that CWD _is_ /proc at the point where:
> 
> +       d = opendir("/proc");

Here cwd does not matter.

> ?  9 lines down in the code we're doing this:
> 
> 		if (chdir(de->d_name) < 0)
> 			continue;
>
> and that will not always do TRT, unless CWD _is_ /proc from the
> beginning.

You are correct, I totally missed this. Fixed now. Thanks.
--
vda
_______________________________________________
busybox mailing list
busybox@busybox.net
http://busybox.net/cgi-bin/mailman/listinfo/busybox
[prev in list] [next in list] [prev in thread] [next in thread]