Hi.

Sorry for late reply.

On Wed, 25 Jul 2007 18:07:58 +0200
Tito  wrote:
> On Tuesday 24 July 2007 02:08:24 Yuichi Nakamura wrote:
> > Hi.
> > 
> 
> snip
> 
> > 
> > About sanitize_path, it is pending.
> > Removing extra slashes is done only there in setfiles.c .
> > Is removing extra slashes done in other applets?
> 
> Yes in devfsd, removing extra slashes at the end of filename:
> 
> in one earlier revision there was:
> 
>  /* strip last / from mount point, so we don't need to check for it later */
>          while (argv[1][1] != '\0' && argv[1][strlen(argv[1]) - 1] == '/')    
>  
>                  argv[1][strlen(argv[1]) - 1] = '\0';
> 
> later it changed to
> 
>          mount_point = bb_simplify_path(argv[1]);
> 
> so far nobody complained.....
I looked at setfiles.c again,
and I found I can use bb_simplify_path, then code gets smaller.
I will use bb_simplify_path.
I will send patch soon.


> > If so, should it be in libbb?
> 
> Could be useful......
> just my 0,02 、
> 
> Ciao,
> Tito
> > I've found bb_simplify_path, but it is expanding "." .
> > 
> > > >
> > > > [snipping the rest of you patch for now]
> > > > HTH,
> > > --
> > > vda
> > 
> > Attached is update.
> 

Regards,
-- 
Yuichi Nakamura
Hitachi Software Engineering Co., Ltd.
Japan SELinux Users Group(JSELUG): http://www.selinux.gr.jp/
SELinux Policy Editor: http://seedit.sourceforge.net/

_______________________________________________
busybox mailing list
[email protected]
http://busybox.net/cgi-bin/mailman/listinfo/busybox

Reply via email to