]> git.ipfire.org Git - thirdparty/samba.git/commit
s3:winbindd: Fix winbindd child logfile name handling
authorPavel Filipenský <pfilipen@redhat.com>
Thu, 7 Oct 2021 10:08:22 +0000 (12:08 +0200)
committerRalph Boehme <slow@samba.org>
Tue, 12 Oct 2021 08:40:34 +0000 (08:40 +0000)
commitb92589c31f0eb3eaf2b3b1867e10b759f6a2edda
treefd1d0c42dfa996af562f581ef2a4f41bd2825ea7
parent4fe965836243928ac33eb95a67d3e889fdc15861
s3:winbindd: Fix winbindd child logfile name handling

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14852

Handling of logfile name for main and child winbindd must ensure:

1) Log directory is selected in this order:
  * -l option of winbindd
  * "log file" parameter in smb.conf
  * compile time value '/usr/local/samba/var'

2) Log filename pattern
  * parent process uses log.winbindd
  * child uses log.wb-<name>

3) Log reopen works for both parent and child (i.e. log filename is not changed)
  * kill -HUP <pid>
  * smbcontrol <pid> reload-config

This commit removes 3 calls of is_default_dyn_LOGFILEBASE() to make sure that:
  - 1st removal: child uses log.wb-<name> after the fork
  - 2nd removal: child after HUP signal, does not switch to log.winbindd
  - 3rd removal: child after smbcontrol reload-config, does not switch to
    log.winbindd

Interesting commits: bfa1b2a8 1484b7f3 3b015a4c d1f7a371

Signed-off-by: Pavel Filipenský <pfilipen@redhat.com>
Reviewed-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
source3/winbindd/winbindd_dual.c