Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 04 Feb 2025 09:11:21 -0800
From:      Cy Schubert <Cy.Schubert@cschubert.com>
To:        Ed Maste <emaste@FreeBSD.org>
Cc:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   Re: git: ecccb638d494 - main - mountd: Promote missing netmask log  to LOG_ERR
Message-ID:  <20250204171121.8EA2C281@slippy.cwsent.com>
In-Reply-To: <202502041658.514Gwo4I049201@gitrepo.freebsd.org>
References:  <202502041658.514Gwo4I049201@gitrepo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
In message <202502041658.514Gwo4I049201@gitrepo.freebsd.org>, Ed Maste 
writes:
> The branch main has been updated by emaste:
>
> URL: https://cgit.FreeBSD.org/src/commit/?id=ecccb638d494f2642f1e2eae8ad932ba
> 70567ccc
>
> commit ecccb638d494f2642f1e2eae8ad932ba70567ccc
> Author:     Ed Maste <emaste@FreeBSD.org>
> AuthorDate: 2025-02-01 13:16:47 +0000
> Commit:     Ed Maste <emaste@FreeBSD.org>
> CommitDate: 2025-02-04 16:58:41 +0000
>
>     mountd: Promote missing netmask log to LOG_ERR
>     
>     Historical network classes are deprecated, but currently still supported
>     in mountd with a LOG_WARNING-priority message added by Mike Karels in
>     2021.  Upgrade the log to LOG_ERR well in advance of this becoming a
>     fatal error.
>     
>     Reviewed by:    rmacklem
>     Sponsored by:   The FreeBSD Foundation
>     Differential Revision: https://reviews.freebsd.org/D48795
> ---
>  usr.sbin/mountd/mountd.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/usr.sbin/mountd/mountd.c b/usr.sbin/mountd/mountd.c
> index 4457a925c2ed..19d76b72da10 100644
> --- a/usr.sbin/mountd/mountd.c
> +++ b/usr.sbin/mountd/mountd.c
> @@ -1806,7 +1806,7 @@ get_exportlist_one(int passno)
>  			len = endcp - cp;
>  		}
>  		if (opt_flags & OP_CLASSMASK)
> -			syslog(LOG_WARNING,
> +			syslog(LOG_ERR,
>  			    "WARNING: No mask specified for %s, "

Should this be changed from WARNING to ERROR too?

>  			    "using out-of-date default",
>  			    (&grp->gr_ptr.gt_net)->nt_name);
>


-- 
Cheers,
Cy Schubert <Cy.Schubert@cschubert.com>
FreeBSD UNIX:  <cy@FreeBSD.org>   Web:  https://FreeBSD.org
NTP:           <cy@nwtime.org>    Web:  https://nwtime.org

			e^(i*pi)+1=0





Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20250204171121.8EA2C281>