Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
References: <1659633958-32756-1-git-send-email-ssengar@linux.microsoft.com> <a939c6f6-1df8-77ac-8366-7121abc2f57b@linux.dev>
In-Reply-To: <a939c6f6-1df8-77ac-8366-7121abc2f57b@linux.dev>
From:   Song Liu <song@kernel.org>
Date:   Tue, 23 Aug 2022 11:07:24 -0700
Message-ID: <CAPhsuW6KRYSXsq-+4ghHuA+-EvBXvtCxYDWYMGwVqbdtFie+bQ@mail.gmail.com>
Subject: Re: [PATCH] md : Replace snprintf with scnprintf
To:     Guoqing Jiang <guoqing.jiang@linux.dev>
Cc:     Saurabh Sengar <ssengar@linux.microsoft.com>,
        Shaohua Li <shli@fb.com>, NeilBrown <neilb@suse.com>,
        linux-raid <linux-raid@vger.kernel.org>,
        open list <linux-kernel@vger.kernel.org>,
        ssengar@microsoft.com, mikelley@microsoft.com
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Thu, Aug 4, 2022 at 6:10 PM Guoqing Jiang <guoqing.jiang@linux.dev> wrot=
e:
>
>
>
> On 8/5/22 1:25 AM, Saurabh Sengar wrote:
> > Current code produces a warning as shown below when total characters
> > in the constituent block device names plus the slashes exceeds 200.
> > snprintf() returns the number of characters generated from the given
> > input, which could cause the expression =E2=80=9C200 =E2=80=93 len=E2=
=80=9D to wrap around
> > to a large positive number. Fix this by using scnprintf() instead,
> > which returns the actual number of characters written into the buffer.
> >
> > [ 1513.267938] ------------[ cut here ]------------
> > [ 1513.267943] WARNING: CPU: 15 PID: 37247 at <snip>/lib/vsprintf.c:250=
9 vsnprintf+0x2c8/0x510
> > [ 1513.267944] Modules linked in:  <snip>
> > [ 1513.267969] CPU: 15 PID: 37247 Comm: mdadm Not tainted 5.4.0-1085-az=
ure #90~18.04.1-Ubuntu
> > [ 1513.267969] Hardware name: Microsoft Corporation Virtual Machine/Vir=
tual Machine, BIOS Hyper-V UEFI Release v4.1 05/09/2022
> > [ 1513.267971] RIP: 0010:vsnprintf+0x2c8/0x510
> > <-snip->
> > [ 1513.267982] Call Trace:
> > [ 1513.267986]  snprintf+0x45/0x70
> > [ 1513.267990]  ? disk_name+0x71/0xa0
> > [ 1513.267993]  dump_zones+0x114/0x240 [raid0]
> > [ 1513.267996]  ? _cond_resched+0x19/0x40
> > [ 1513.267998]  raid0_run+0x19e/0x270 [raid0]
> > [ 1513.268000]  md_run+0x5e0/0xc50
> > [ 1513.268003]  ? security_capable+0x3f/0x60
> > [ 1513.268005]  do_md_run+0x19/0x110
> > [ 1513.268006]  md_ioctl+0x195e/0x1f90
> > [ 1513.268007]  blkdev_ioctl+0x91f/0x9f0
> > [ 1513.268010]  block_ioctl+0x3d/0x50
> > [ 1513.268012]  do_vfs_ioctl+0xa9/0x640
> > [ 1513.268014]  ? __fput+0x162/0x260
> > [ 1513.268016]  ksys_ioctl+0x75/0x80
> > [ 1513.268017]  __x64_sys_ioctl+0x1a/0x20
> > [ 1513.268019]  do_syscall_64+0x5e/0x200
> > [ 1513.268021]  entry_SYSCALL_64_after_hwframe+0x44/0xa9
> >
> > Fixes: 766038846e875 ("md/raid0: replace printk() with pr_*()")
> > Signed-off-by: Saurabh Sengar <ssengar@linux.microsoft.com>

The patch doesn't apply. Could you please rebase and resend?

Thanks,
Song

> > ---
> >   drivers/md/raid0.c | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/md/raid0.c b/drivers/md/raid0.c
> > index b21e101..b6f0fc1 100644
> > --- a/drivers/md/raid0.c
> > +++ b/drivers/md/raid0.c
> > @@ -48,7 +48,7 @@ static void dump_zones(struct mddev *mddev)
> >               int len =3D 0;
> >
> >               for (k =3D 0; k < conf->strip_zone[j].nb_dev; k++)
> > -                     len +=3D snprintf(line+len, 200-len, "%s%s", k?"/=
":"",
> > +                     len +=3D scnprintf(line+len, 200-len, "%s%s", k?"=
/":"",
> >                                       bdevname(conf->devlist[j*raid_dis=
ks
> >                                                              + k]->bdev=
, b));
> >               pr_debug("md: zone%d=3D[%s]\n", j, line);
>
> Acked-by: Guoqing Jiang <guoqing.jiang@linux.dev>
>
> Thanks,
> Guoqing