linux-kernel - Re: [RESEND PATCH] cifs: Use str_yes_no() helper in cifs_ses_add

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAH2r5mvDEcu_-QkS=e-sUx3-_OAbz93P=pO7M-7B_aatKfRbrw@mail.gmail.com>
Date: Mon, 9 Dec 2024 15:25:31 -0600
From: Steve French <smfrench@...il.com>
To: Thorsten Blum <thorsten.blum@...ux.dev>
Cc: Steve French <sfrench@...ba.org>, Paulo Alcantara <pc@...guebit.com>, 
	Ronnie Sahlberg <ronniesahlberg@...il.com>, Shyam Prasad N <sprasad@...rosoft.com>, 
	Tom Talpey <tom@...pey.com>, Bharath SM <bharathsm@...rosoft.com>, linux-cifs@...r.kernel.org, 
	samba-technical@...ts.samba.org, linux-kernel@...r.kernel.org
Subject: Re: [RESEND PATCH] cifs: Use str_yes_no() helper in cifs_ses_add_channel()

merged into cifs-2.6.git for-next

On Mon, Dec 9, 2024 at 5:10 AM Thorsten Blum <thorsten.blum@...ux.dev> wrote:
>
> Remove hard-coded strings by using the str_yes_no() helper function.
>
> Signed-off-by: Thorsten Blum <thorsten.blum@...ux.dev>
> ---
>  fs/smb/client/sess.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/fs/smb/client/sess.c b/fs/smb/client/sess.c
> index 0bb77f9ec686..3306fb655136 100644
> --- a/fs/smb/client/sess.c
> +++ b/fs/smb/client/sess.c
> @@ -488,11 +488,11 @@ cifs_ses_add_channel(struct cifs_ses *ses,
>
>         if (iface->sockaddr.ss_family == AF_INET)
>                 cifs_dbg(FYI, "adding channel to ses %p (speed:%zu bps rdma:%s ip:%pI4)\n",
> -                        ses, iface->speed, iface->rdma_capable ? "yes" : "no",
> +                        ses, iface->speed, str_yes_no(iface->rdma_capable),
>                          &ipv4->sin_addr);
>         else
>                 cifs_dbg(FYI, "adding channel to ses %p (speed:%zu bps rdma:%s ip:%pI6)\n",
> -                        ses, iface->speed, iface->rdma_capable ? "yes" : "no",
> +                        ses, iface->speed, str_yes_no(iface->rdma_capable),
>                          &ipv6->sin6_addr);
>
>         /*
> --
> 2.47.1
>
>


-- 
Thanks,

Steve