Page MenuHomeFreeBSD

sshd: remove unneeded initialization of libwrap logging severities
ClosedPublic

Authored by glebius on May 11 2023, 7:11 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, May 21, 8:27 PM
Unknown Object (File)
Feb 28 2024, 6:01 PM
Unknown Object (File)
Feb 3 2024, 8:40 PM
Unknown Object (File)
Dec 22 2023, 11:51 PM
Unknown Object (File)
Dec 18 2023, 5:42 PM
Unknown Object (File)
Dec 12 2023, 1:04 AM
Unknown Object (File)
Nov 28 2023, 4:55 PM
Unknown Object (File)
Nov 22 2023, 8:20 AM
Subscribers

Details

Summary

This part of ca573c9a177 proved to be unnecessary. As the removed
comment says, we set them merely for logging syntax errors, as we log
refusals ourselves. However, inside the libwrap the parser logs any
syntax errors with tcpd_warn() which has hardcoded LOG_WARNING inside.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

philip added a subscriber: philip.

If neither allow_severity or deny_severity are referenced anywhere else, this looks good to me.
I have only reviewed the patch. I haven't actually tested it.

crypto/openssh/sshd.c
145

Is allow_severity not referenced anywhere else?

This revision is now accepted and ready to land.May 16 2023, 6:06 AM