[cifs-protocol] [EXTERNAL] MS-SMB2: Reconnect of second disconnected PH on a file fails if contending open broke H lease - TrackingID#2509250040006209

[Ralph Boehme]

Hi Obaid,

https://gitlab.com/samba-team/samba/-/merge_requests/4188

$ git fetch "git at gitlab.com:samba-team/devel/samba.git" 'slow/ph-tests'
$ git checkout -b 'samba-slow/ph-tests' FETCH_HEAD

The test:

$ bin/smbtorture -d 0 -U 'USER%PASSWORD' //SERVER/SHARE 
smb2.persistent-open.reconnect-contended-two --option=torture:single=yes 
--option=torture:subtest=17

Server must be Fileserver-For-General-Use.

Thanks!

On 10/4/25 12:36 AM, Obaid Farooqi wrote:
> Hi Ralph:
> Is this one of the PH test cases that I can run?
> Please let me know the test case # and, also upload a network capture of this scenario to the following link:
> 
>   https://support.microsoft.com/files?workspace=eyJhbGciOiJSUzI1NiIsImtpZCI6IjJBNzk1QUQxMDNDQTM4OEZENEQzREQxQTZERkU4QTE2RDkyMkNDQkMiLCJ0eXAiOiJKV1QifQ.eyJ3c2lkIjoiNzUzYTIwMzktY2Q3OS00ZWZjLTg0YmYtZTIyYzIzZDQyNzE2Iiwic3IiOiIyNTA5MjUwMDQwMDA2MjA5Iiwic3YiOiJ2MSIsInJzIjoiRXh0ZXJuYWwiLCJ3dGlkIjoiNjRhMzY4NTMtNjdmYS00NWQ3LTg5NjItMjc0ZGZiZGViYWRhIiwiYXBwaWQiOiI0ZTc2ODkxZC04NDUwLTRlNWUtYmUzOC1lYTNiZDZlZjIxZTUiLCJuYmYiOjE3NTk1MzA4OTYsImV4cCI6MTc2NzMwNjg5NSwiaWF0IjoxNzU5NTMwODk2LCJpc3MiOiJodHRwczovL2FwaS5kdG1uZWJ1bGEubWljcm9zb2Z0LmNvbSIsImF1ZCI6Imh0dHA6Ly9zbWMifQ.J0gP8LIQEThWlwlLlFsh6XIgtWNkc4vnRFxWjaVFXmmWzFbkUXuT647F4sRMgnYP6CWx27weuZ9Vo7LdUqS1lupAXlJBeHlzq5sEsvA8Xl1H_gsLrGMJROpJl1aE9GUBzjrypWjB6wI0WZc99JuiQ3eEW9-aVlauQOziwvzm_5DYd8SQ9Cje91qnBhR2Zoy7wwK3laKh_-WtCHe1xkWCuXV7ywV2tSwdvnPrCJLiQObWZxailC1xgaRQ9gvSL1_PAqCfT98bOkuCPdX6VlJ5Va7DpBPd24dZCdUlJxjOs12pwgcTP1Ibz_G0DrxMqr7-VUI62XhfktAQIzABLgYnwg&wid=753a2039-cd79-4efc-84bf-e22c23d42716
> 
> Regards,
> Obaid Farooqi
> Sr. Escalation Engineer | Microsoft
> 
> -----Original Message-----
> From: Michael Bowen <Mike.Bowen at microsoft.com>
> Sent: Thursday, September 25, 2025 10:07 AM
> To: Ralph Boehme <slow at samba.org>
> Cc: cifs-protocol at lists.samba.org; Microsoft Support <supportmail at microsoft.com>
> Subject: RE: [EXTERNAL] MS-SMB2: Reconnect of second disconnected PH on a file fails if contending open broke H lease - TrackingID#2509250040006209
> 
> Hi Ralph,
> 
> Thanks for your question. I've created case number 2509250040006209to track this issue. Please leave the number in the subject line and use reply all your correspondence. One of our engineers will contact you soon.
> 
> Best regards,
> Michael Bowen
> Sr. Escalation Engineer - Microsoft® Corporation
>   
> 
> 
> -----Original Message-----
> From: Ralph Boehme <slow at samba.org>
> Sent: Thursday, September 25, 2025 7:46 AM
> To: Interoperability Documentation Help <dochelp at microsoft.com>
> Cc: cifs-protocol at lists.samba.org
> Subject: [EXTERNAL] MS-SMB2: Reconnect of second disconnected PH on a file fails if contending open broke H lease
> 
> Hi dochelp!
> 
> I'm sorry, but I just ran into another issue I fail to understand... :)
> 
> Scenario:
> 
> Reconnect of second disconnected PH on a file fails if contending open caused a break of the H lease.
> 
> Server: Windows 2025, Fileserver for General Use
> Client: smbtorture
> 
> CG := Create Guid
> LK := Lease Key
> 
> 1. Client 1: open file with PH, lease=RWH, LK=X, CG=Y 2. Client 1: another open on the same file with PH, lease=RWH, LK=X, CG=Z (same LK as in 1) 3. Disconnect 4. Client 2: content disconnected handles with another open that triggers a break and is suspended 5. Client 1 reconnects 6. Client 1 reconnects first PH 7. Server sends lease break for LK=X to NONE to client 1 8. Client 1 acks lease break 9. Client 1 attempts to reconnect second PH but this fails with STATUS_FILE_NOT_AVAILABLE
> 
> In my understanding 9 should not fail and in my implementation of PH in Samba it succeeds.
> 
> Am I missing anything in the docs why 9 should fail?
> 
> In my testing with various lease levels and different sharemodes and access masks of the contending open, the pattern seems to be that the second reconnect alway fails if the contending open triggered a H lease break.
> 
> Thanks!
> -slow
> 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature.asc
Type: application/pgp-signature
Size: 840 bytes
Desc: OpenPGP digital signature
URL: <http://lists.samba.org/pipermail/cifs-protocol/attachments/20251006/fe8b5b23/OpenPGP_signature.sig>