samba - Nov 2019 - get_share_mode_lock:, get_static_share_mode_data failed: NT_STATUS_NO_MEMORY with Samba 4.11.2

Hello all,
after the upgarde from Samba 4.10.7 to 4.11.2 we get lots of these in
our logfiles:


2019-11-28T20:40:44+01:00 lx-sv-09 smbd_audit: [2019/11/28
20:40:44.886615,  1]
../../source3/locking/share_mode_lock.c:597(get_share_mode_lock)
2019-11-28T20:40:44+01:00 lx-sv-09 smbd_audit:  get_share_mode_lock:
get_static_share_mode_data failed: NT_STATUS_NO_MEMORY

There are no symptoms accept those errors in the logs. At least none
were reported until now.
Has anybody seen this before?

System is centos7 with the sernet packages running as a domain member.

here is the smb.conf (I know about the idmap range. It was a classic
upgrade)

[global]
        security = ADS
        realm = AD.DOM.COM
        log level = 1 auth_audit:3
        logging = syslog only
        workgroup = AD-02
        netbios name = LX-SV-09
        server string = Daten
        kerberos method = secrets and keytab

##Network##
        interfaces = lo eth0
        bind interfaces only = Yes
        writable = yes
        disable netbios = yes
        smb ports = 445
        server min protocol = SMB2
        client min protocol = SMB2

##acl stuff##
        map acl inherit = YES
        store dos attributes = yes
        username map = /etc/samba/smbusers
        inherit acls = Yes

##Printers##
        load printers = no
        printing = bsd
        printcap name = /dev/null
        disable spoolss = yes

##idmap##
        # Default idmap config used for BUILTIN and local windows
accounts/groups
        idmap config *:backend = tdb
        idmap config *:range = 3000000-3100000

        # idmap config for domain AD-02
        idmap config AD-02:backend = ad
        idmap config AD-02:range = 500-65555
        idmap config AD-02:schema_mode = rfc2307
        idmap config AD-02:unix_nss_info = yes
        winbind use default domain = Yes
        winbind offline logon = yes
        winbind refresh tickets = yes
        machine password timeout = 0
##audit

vfs objects = acl_xattr full_audit shadow_copy2
full_audit:failure = none
full_audit:success = pwrite write rename unlink
full_audit:prefix = IP=%I|USER=%u|MACHINE=%m|VOLUME=%S
full_audit:facility = local7
full_audit:priority = NOTICE




There was a post previously with the same Error message but I think no
real solution.

https://www.spinics.net/lists/samba/msg160882.html


Regards


Christian

-- 
Dr. Christian Naumer
Unit Head Bioprocess Development
B.R.A.I.N Aktiengesellschaft
Darmstaedter Str. 34-36, D-64673 Zwingenberg
e-mail cn at brain-biotech.com, homepage www.brain-biotech.com
fon +49-6251-9331-30  /   fax +49-6251-9331-11

Sitz der Gesellschaft: Zwingenberg/Bergstrasse
Registergericht AG Darmstadt, HRB 24758
Vorstand: Dr. Juergen Eck (Vorsitzender), Manfred Bender,
Ludger Roedder
Aufsichtsratsvorsitzender: Dr. Georg Kellinghusen

On 29/11/2019 08:04, Christian Naumer via samba wrote:
> Hello all,
> after the upgarde from Samba 4.10.7 to 4.11.2 we get lots of these in
> our logfiles:
>
>
> 2019-11-28T20:40:44+01:00 lx-sv-09 smbd_audit: [2019/11/28
> 20:40:44.886615,  1]
> ../../source3/locking/share_mode_lock.c:597(get_share_mode_lock)
> 2019-11-28T20:40:44+01:00 lx-sv-09 smbd_audit:  get_share_mode_lock:
> get_static_share_mode_data failed: NT_STATUS_NO_MEMORY
>
> There are no symptoms accept those errors in the logs.
There wouldn't be, it isn't an error ;-)

If you look at '/source3/locking/share_mode_lock.c'
'get_share_mode_lock'

The relevant part is this:

 ??? ??? status = get_static_share_mode_data(
 ??? ??? ??? static_share_mode_record,
 ??? ??? ??? id,
 ??? ??? ??? servicepath,
 ??? ??? ??? smb_fname,
 ??? ??? ??? old_write_time);
 ??? ??? if (!NT_STATUS_IS_OK(status)) {
 ??? ??? ??? DBG_WARNING("get_static_share_mode_data failed: %s\n",
 ??? ??? ??? ??? ??? nt_errstr(status));
 ??? ??? ??? goto fail;
 ??? ??? }

It calls 'get_static_share_mode_data' and the relevant part of that is
this:

 ??? if (value.dptr == NULL) {
 ??? ??? d = fresh_share_mode_lock(
 ??? ??? ??? lock_db, servicepath, smb_fname, old_write_time);
 ??? ??? if (d == NULL) {
 ??? ??? ??? return NT_STATUS_NO_MEMORY;
 ??? ??? }

Which I hope you can see, is saying that it cannot find the share 
record, this isn't an actual error.

It has also been fixed in Samba git, see here:

https://git.samba.org/?p=samba.git;a=commitdiff;h=79e3b1c71f59591c54e87299984e50d2ffb00b6b

So, you can just ignore them ;-)

Rowland

Am 29.11.19 um 10:24 schrieb Rowland penny via samba:
> 
> Which I hope you can see, is saying that it cannot find the share
> record, this isn't an actual error.
> 
> It has also been fixed in Samba git, see here:
> 
>
https://git.samba.org/?p=samba.git;a=commitdiff;h=79e3b1c71f59591c54e87299984e50d2ffb00b6b
> 
> 
> So, you can just ignore them ;-)
> 

Thanks!


> Rowland
> 
> 
> 
-- 
Dr. Christian Naumer
Unit Head Bioprocess Development
B.R.A.I.N Aktiengesellschaft
Darmstaedter Str. 34-36, D-64673 Zwingenberg
e-mail cn at brain-biotech.com, homepage www.brain-biotech.com
fon +49-6251-9331-30  /   fax +49-6251-9331-11

Sitz der Gesellschaft: Zwingenberg/Bergstrasse
Registergericht AG Darmstadt, HRB 24758
Vorstand: Dr. Juergen Eck (Vorsitzender), Manfred Bender,
Ludger Roedder
Aufsichtsratsvorsitzender: Dr. Georg Kellinghusen

Hai, 

I'm wondering, why this is not in 4.11 ? 
The current error messages, makes people look in the wrong direction. 

Do you know if its prepaired it for next release? Else i'll patch it into my
builds.

Rowland, tell me, how did you find the change through git, thats still a part i
need to improve. :-/
I looked in bugzilla, but was not able to find it. 


Thanks, i noticed this one also.

Greetz, 

Louis



> -----Oorspronkelijk bericht-----
> Van: samba [mailto:samba-bounces at lists.samba.org] Namens 
> Rowland penny via samba
> Verzonden: vrijdag 29 november 2019 10:24
> Aan: samba at lists.samba.org
> Onderwerp: Re: [Samba] get_share_mode_lock:, 
> get_static_share_mode_data failed: NT_STATUS_NO_MEMORY with 
> Samba 4.11.2
> 
> On 29/11/2019 08:04, Christian Naumer via samba wrote:
> > Hello all,
> > after the upgarde from Samba 4.10.7 to 4.11.2 we get lots 
> of these in
> > our logfiles:
> >
> >
> > 2019-11-28T20:40:44+01:00 lx-sv-09 smbd_audit: [2019/11/28
> > 20:40:44.886615,  1]
> > ../../source3/locking/share_mode_lock.c:597(get_share_mode_lock)
> > 2019-11-28T20:40:44+01:00 lx-sv-09 smbd_audit:  get_share_mode_lock:
> > get_static_share_mode_data failed: NT_STATUS_NO_MEMORY
> >
> > There are no symptoms accept those errors in the logs.
> 
> There wouldn't be, it isn't an error ;-)
> 
> If you look at '/source3/locking/share_mode_lock.c' 
> 'get_share_mode_lock'
> 
> The relevant part is this:
> 
>  ??? ??? status = get_static_share_mode_data(
>  ??? ??? ??? static_share_mode_record,
>  ??? ??? ??? id,
>  ??? ??? ??? servicepath,
>  ??? ??? ??? smb_fname,
>  ??? ??? ??? old_write_time);
>  ??? ??? if (!NT_STATUS_IS_OK(status)) {
>  ??? ??? ??? DBG_WARNING("get_static_share_mode_data failed:
%s\n",
>  ??? ??? ??? ??? ??? nt_errstr(status));
>  ??? ??? ??? goto fail;
>  ??? ??? }
> 
> It calls 'get_static_share_mode_data' and the relevant part 
> of that is this:
> 
>  ??? if (value.dptr == NULL) {
>  ??? ??? d = fresh_share_mode_lock(
>  ??? ??? ??? lock_db, servicepath, smb_fname, old_write_time);
>  ??? ??? if (d == NULL) {
>  ??? ??? ??? return NT_STATUS_NO_MEMORY;
>  ??? ??? }
> 
> Which I hope you can see, is saying that it cannot find the share 
> record, this isn't an actual error.
> 
> It has also been fixed in Samba git, see here:
> 
> https://git.samba.org/?p=samba.git;a=commitdiff;h=79e3b1c71f59
> 591c54e87299984e50d2ffb00b6b
> 
> So, you can just ignore them ;-)
> 
> Rowland
> 
> 
> 
> -- 
> To unsubscribe from this list go to the following URL and read the
> instructions:  https://lists.samba.org/mailman/options/samba
> 
>

On 29/11/2019 10:51, L.P.H. van Belle via samba wrote:
> Hai,
>
> I'm wondering, why this is not in 4.11 ?
> The current error messages, makes people look in the wrong direction.
>
> Do you know if its prepaired it for next release? Else i'll patch it
into my builds.
>
> Rowland, tell me, how did you find the change through git, thats still a
part i need to improve. :-/
> I looked in bugzilla, but was not able to find it.
If you check the log fragment the OP provided:

2019-11-28T20:40:44+01:00 lx-sv-09 smbd_audit: [2019/11/28 
20:40:44.886615, 
1]../../source3/locking/share_mode_lock.c:597(get_share_mode_lock)

It tells you where to search in Samba git: 
https://git.samba.org/?p=samba.git;a=tree;hb=HEAD

So, if you follow the path provided:

source3 --> locking --> share_mode_lock.c

You can then open 'share_mode_lock.c' in your favourite editor and go to
'get_share_mode_lock' and then look for the error message, in this case 
'get_static_share_mode_data failed'. This also returns
'nt_errstr' from
'status' and this comes from 'status =
get_static_share_mode_data(' . In
this case, it returns 'NT_STATUS_NO_MEMORY'.

Now you know where it comes from, you can check the history for 
'share_mode_lock.c', this is where I found this:

2019-07-17 ??? Volker Lendecke??? smbd: Increase a debug level

Rowland