Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 16 Dec 2024 16:24:15 +0100
From:      Baptiste Daroussin <bapt@freebsd.org>
To:        Olivier Certner <olce@freebsd.org>
Cc:        src-committers@freebsd.org, dev-commits-src-all@freebsd.org,  dev-commits-src-main@freebsd.org
Subject:   Re: git: d2be7ed63aff - main - cred: proc_set_cred(), proc_unset_cred(): Update user's process count
Message-ID:  <hknkwaw3gs27njpczq7cri62tinpiefks2wfjbp2t3il7xl2fc@36fwjkyh22zx>
In-Reply-To: <202412161445.4BGEjYvf052320@gitrepo.freebsd.org>
References:  <202412161445.4BGEjYvf052320@gitrepo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon 16 Dec 14:45, Olivier Certner wrote:
> The branch main has been updated by olce:
> 
> URL: https://cgit.FreeBSD.org/src/commit/?id=d2be7ed63affd8af5fe6203002b7cc3cbe7f7891
> 
> commit d2be7ed63affd8af5fe6203002b7cc3cbe7f7891
> Author:     Olivier Certner <olce@FreeBSD.org>
> AuthorDate: 2024-08-02 15:57:51 +0000
> Commit:     Olivier Certner <olce@FreeBSD.org>
> CommitDate: 2024-12-16 14:42:32 +0000
> 
>     cred: proc_set_cred(), proc_unset_cred(): Update user's process count
>     

[snip]

> +
> +void
> +proc_set_cred(struct proc *p, struct ucred *newcred)
> +{
> +	bool success = _proc_set_cred(p, newcred, false);


This breaks build for GERNERIC-NODEBUG

--- kern_prot.o ---
/home/pkgbuild/worktrees/main/sys/kern/kern_prot.c:2697:7: error: unused
variable 'success' [-Werror,-Wunused-variable]
 2697 |         bool success = _proc_set_cred(p, newcred, false);
       |              ^~~~~~~

> +
> +	MPASS(success);
> +}
> +

Best regards,
Bapt



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