[cifs-protocol] [REG:116022413751567 ] Response to SVHDX_OPEN_DEVICE_CONTEXT[_V2] when shared VHDX is not supported
bburgin at microsoft.com
Thu Feb 25 22:02:25 UTC 2016
The errata only concerns processing for create context names that are unknown (like 'AAPL'). [MS-SMB2] 184.108.40.206 "SMB2_CREATE_CONTEXT Request Values" enumerates the known context names and identifies 0x9CCBCF9E04C1E643980E158DA1F6EC83 as SVHDX_OPEN_DEVICE_CONTEXT. [MS-SMB2] further has the server processing rule for at 220.127.116.11.14 "Handling the SVHDX_OPEN_DEVICE_CONTEXT Create Context" that describe "If IsSharedVHDSupported is FALSE, the server MUST fail the request with STATUS_INVALID_DEVICE_REQUEST" for "servers that implement the SMB 3.0.2 or SMB 3.1.1 dialect".
From: Jeremy Allison [mailto:jra at samba.org]
Sent: Thursday, February 25, 2016 1:50 PM
To: Bryan Burgin <bburgin at microsoft.com>
Cc: Uri Simchoni <uri at samba.org>; cifs-protocol at lists.samba.org; MSSolve Case Email <casemail at microsoft.com>
Subject: Re: [cifs-protocol] [REG:116022413751567 ] Response to SVHDX_OPEN_DEVICE_CONTEXT[_V2] when shared VHDX is not supported
On Thu, Feb 25, 2016 at 08:54:56PM +0000, Bryan Burgin wrote:
> However, per the spec, we believe that you should be either supporting
> the context (and RSVD) or failing the Create.
Hang on, the errata now says:
In section 18.104.22.168 Receiving an SMB2 CREATE Request, the first paragraph under Create Context Validation has been changed.
The server SHOULD<247> fail any request having a create context not specified in section 22.214.171.124 with a STATUS_INVALID_PARAMETER error.
<247> Section 126.96.36.199: Windows Vista SP1, Windows Server 2008, Windows 7, Windows Server 2008 R2, Windows 8, and Windows Server 2012 ignore create contexts having a NameLength greater than 4 and ignore create contexts with a length of 4 that are not specified in section 188.8.131.52.
The server MUST fail create contexts having a NameLength less than 4 with a STATUS_INVALID_PARAMETER error.
That doesn't say anything about supporting a specific context - just return STATUS_INVALID_PARAMETER if NameLength < 4 - which we already do.
As far as I can see we are correctly implementing the spec.
More information about the cifs-protocol