[PATCH] fix VFS MACROS again...

Stefan (metze) Metzmacher metze at metzemix.de
Fri Jun 6 06:15:49 GMT 2003


At 05:58 06.06.2003 +0000, jra at dp.samba.org wrote:
>On Fri, Jun 06, 2003 at 07:55:51AM +0200, Stefan (metze) Metzmacher wrote:
> > Hi Jeremy,
> >
> > your EA patch introduces bugs into vfs_macros.h
> >
> > the SMB_VFS_NEXT_* macros are not only cut-n-paste!
> >
> > Please be a bit more careful (so close to release)
> >
> > The best is to add new vfs functions to the skel_*.c modules
> > and test if it compiles fine.
>
>So can you explain exactly what the differences are
>in the VFS_NEXT - it's not listed in the include file
>and is not obvious.

for the SMB_VFS_* and SMB_VFS_OPAQUE_*

it's allways rename "SMB_VFS_ " to "SMB_VFS_OPAQUE_"
and "->vfs."  to "->vfs_opaque."

but for SMB_VFS_NEXT_*

you need to take care of the function which got fsp not conn.

so (handle)->conn->vfs_next doesn't exist.
it's just always (handle)->vfs_next.

(like in the patch I sended to you :-)


>We need this (EA) functionality to do the ACLs right,
>so I'd rather annoy the Samba developers now, than the
>customers later :-).
>
>Jeremy.


metze
-----------------------------------------------------------------------------
Stefan "metze" Metzmacher <metze at metzemix.de> 




More information about the samba-technical mailing list