Patch: Fixed patch for moving RSVD/SHVXD to vfs_default

Richard Sharpe realrichardsharpe at
Wed Aug 5 20:41:38 UTC 2015

On Wed, Aug 5, 2015 at 11:42 AM, Michael Adam <obnox at> wrote:
> On 2015-08-04 at 21:56 +0200, Ralph Böhme wrote:
>> On Tue, Aug 04, 2015 at 12:06:38PM -0700, Richard Sharpe wrote:
>> > On Tue, Aug 4, 2015 at 11:25 AM, Michael Adam <obnox at> wrote:
>> > > Wait a minute ... the vfs should not be fiddling
>> > > with smb2 create contexts! IMHO, this belongs into
>> > > smb2_create.c and nowhere else.
>> >
>> > Why? These are the same as Extra Create Parameters in Windows and they
>> > belong in NTFS, AFAICT.
>> >
>> > In addition, we already pass these to the VFS layer, and why should we
>> > have to add code to the SMB layer for each new ECP/Create Context
>> > Microsoft adds?
>> iirc it was me who added the create contexts to
>> SMB_VFS_CREATE_FILE(). Makes sense to me.
> Oh, I didn't realize that. This is awful from a software layering
> point of view (just in my humble and possibly naive opinion...).
> These create contexts belong to the SMB level. The VFS represents
> the virtual NT file system layer. Some of the create contexts may
> represent things that need to have an effect to the vfs layer.
> In that case, I'd say we need to provide something special.

I think you are wrong here. Create Contexts are mapped from Extra
Create Paramaters on Windows, and they can come from userland on

Here is a list of ECPs that originate above the redirector:

Notice that last one!

Richard Sharpe

More information about the samba-technical mailing list