CVS update: samba/source/smbd

Simo Sorce simo.sorce at xsec.it
Tue Aug 27 20:40:08 EST 2002


give me time :))
I had a very similar pacth in my queue with lp_unhidespecial option.
Also I changed the hide veto files sematics on purpose!

however, if you are happy with this one I'm fine, I only do not like at
all the fact that you have dependencies on the stat and name created by
previous functions, It asks for bugs.

If you do not know that you can fuck the pst structure by later changes
in functions and the second call will not work, the same is for name.

do you have any objections if I movr back outside that functions the
stat? (Their place is really outside that functions imho)

Also making a new function call only to do 3 ifs seem to much to me.

Simo.

On Mon, 2002-08-26 at 18:38, jra at samba.org wrote:
> 
> Date:	Mon Aug 26 12:38:08 2002
> Author:	jra
> 
> Update of /data/cvs/samba/source/smbd
> In directory dp.samba.org:/tmp/cvs-serv11244/smbd
> 
> Modified Files:
> 	dir.c 
> Log Message:
> Reverted and tidied up the "special" files patch. Adds "hide special files"
> option (set to false by default). Made checking for hide unreadable/writable
> more efficient (less stat calls, less allocating printf calls).
> Jeremy.
> 
> 
> Revisions:
> dir.c		1.72 => 1.73
> 	http://www.samba.org/cgi-bin/cvsweb/samba/source/smbd/dir.c?r1=1.72&r2=1.73
-- 
Simo Sorce - simo.sorce at xsec.it
Xsec s.r.l.
via Durando 10 Ed. G - 20158 - Milano
tel. +39 02 2399 7130 - fax: +39 02 700 442 399
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 232 bytes
Desc: This is a digitally signed message part
Url : http://lists.samba.org/archive/samba-cvs/attachments/20020827/f21d52c7/attachment.bin


More information about the samba-cvs mailing list