#2493 Check chown_debug_file() usage
Closed: Fixed Opened by sbose.

Currently chown_debug_file() is called unconditionally and it does not make any check if debug to files is active or not. This might cause irritating error messages e.g. when journald is used.


Fields changed

rhbz: => 0

Fields changed

milestone: NEEDS_TRIAGE => SSSD 1.12.4

Moving tickets that didn't make the 1.12.4 release to 1.12.5

milestone: SSSD 1.12.4 => SSSD 1.12.5

Fields changed

milestone: SSSD 1.12.5 => SSSD 1.13.2
owner: jhrozek => pcech
sensitive: => 0

Fields changed

patch: 0 => 1

Fields changed

status: new => assigned

To reproduce this you should start sssd without the '-f' option and /var/log/sssd should either empty or not exist. If you then check the journal you you see something like

# journalctl -r | grep 'chown failed'
Okt 23 13:54:57 sssd-devel.sssd.devel sssd_ifp[4963]: chown failed for [sssd_ifp]: [2]
Okt 23 13:54:57 sssd-devel.sssd.devel sssd_pam[4962]: chown failed for [sssd_pam]: [2]
Okt 23 13:54:57 sssd-devel.sssd.devel sssd_nss[4961]: chown failed for [sssd_nss]: [2]
Okt 23 13:54:57 sssd-devel.sssd.devel sssd_be[4960]: chown failed for [sssd_ad.devel]: [2]
Okt 23 13:54:57 sssd-devel.sssd.devel sssd_be[4960]: chown failed for [sssd_ad.devel]: [2]

master:

  • 152fed23797c8950ca18cf6dc2bddb61a3f615c8

sssd-1-13:

  • 6e2822b151c21ce6e3287a0cf25d40e9f10a6127

Fields changed

resolution: => fixed
status: assigned => closed

master:

  • 74cc3797e311fd5dc12473678f499041ade7784f

sssd-1-13:

  • cbb04b9439d73fff027e193093a49cdac3cdb499

Metadata Update from @sbose:
- Issue assigned to pcech
- Issue set to the milestone: SSSD 1.13.2

SSSD is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in SSSD's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/SSSD/sssd/issues/3535

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Metadata