Add smbstatus test (lock file with smbclient)

Ralph Böhme slow at samba.org
Wed Mar 13 15:16:13 UTC 2019



> Am 13.03.2019 um 15:53 schrieb Andreas Schneider <asn at samba.org>:
> 
> On Wednesday, March 13, 2019 3:02:25 PM CET Ralph Böhme wrote:
>> Hi Andreas
>> 
>> Am 13.03.2019 um 12:08 schrieb Andreas Schneider via samba-technical <samba-
> technical at lists.samba.org>:
>>> I've created a patch for 'smbstatus -L --resolve-uids' to show usernames
>>> instead of UIDs as I have a customer request for it.
>>> 
>>> However Andrew rejected the patch to go in without a test. I've tried to
>>> implement a test for smbstatus but it doesn't work. My problem is that I'm
>>> not able to create a lock on a file to show up with smbstatus later.
>>> 
>>> I can:
>>> 
>>> open <file>
>>> 
>>> but I can't pass the returned fnum to
>>> 
>>> lock <fnum> r 0x0 0x1
>> 
>> Looks like lock is obly implemented for SMB1. Also, you seem to use a
>> hardcoded fnum value of 0, for SMB2+ I seem to get 1, and for SMB1 it's
>> worse, as you'll get a random number there.
> 
> I just put 0 there to have a number, you need to pass the one you get from 
> open which I can't.

You don't need a brl-lock to test smbstatus -L, all you need is an open. Or am I missing something?


> So with SMB2+ it would be more reliable, but we don't have 
> Unix extensions yet (*hint*)!

Hm, why do we only support POSIX brl in smbclient? SMB support byte-range-locks out-of the box, no need for UNIX extensions here. Jeremy? 

-slow


More information about the samba-technical mailing list