linux-kernel - RE: [PATCH net-next v4 3/4] net: renesas: rswitch: add offloading for L2 switching

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] [thread-next>] [day] [month] [year] [list]

Message-ID:
 <TY4PR01MB1428244B42D6CE0A5BD136BFF823AA@TY4PR01MB14282.jpnprd01.prod.outlook.com>
Date: Fri, 29 Aug 2025 06:15:05 +0000
From: Michael Dege <michael.dege@...esas.com>
To: ALOK TIWARI <alok.a.tiwari@...cle.com>, Yoshihiro Shimoda
	<yoshihiro.shimoda.uh@...esas.com>, niklas.soderlund
	<niklas.soderlund@...natech.se>, Paul Barker <paul@...rker.dev>, Andrew Lunn
	<andrew+netdev@...n.ch>, "David S. Miller" <davem@...emloft.net>, Eric
 Dumazet <edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>, Paolo Abeni
	<pabeni@...hat.com>
CC: "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
	"linux-renesas-soc@...r.kernel.org" <linux-renesas-soc@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, Nikita
 Yushchenko <nikita.yoush@...entembedded.com>
Subject: RE: [PATCH net-next v4 3/4] net: renesas: rswitch: add offloading for
 L2 switching

Hello Alok,

> -----Original Message-----
> From: ALOK TIWARI <alok.a.tiwari@...cle.com>
> Sent: Thursday, August 28, 2025 8:42 PM
> To: Michael Dege <michael.dege@...esas.com>; Yoshihiro Shimoda <yoshihiro.shimoda.uh@...esas.com>;
> niklas.soderlund <niklas.soderlund@...natech.se>; Paul Barker <paul@...rker.dev>; Andrew Lunn
> <andrew+netdev@...n.ch>; David S. Miller <davem@...emloft.net>; Eric Dumazet <edumazet@...gle.com>;
> Jakub Kicinski <kuba@...nel.org>; Paolo Abeni <pabeni@...hat.com>
> Cc: netdev@...r.kernel.org; linux-renesas-soc@...r.kernel.org; linux-kernel@...r.kernel.org; Nikita
> Yushchenko <nikita.yoush@...entembedded.com>
> Subject: Re: [PATCH net-next v4 3/4] net: renesas: rswitch: add offloading for L2 switching
>
>
>
> On 8/28/2025 3:53 PM, Michael Dege wrote:
> > +static int rswitch_get_port_parent_id(struct net_device *ndev,
> > +                                 struct netdev_phys_item_id *ppid) {
> > +   struct rswitch_device *rdev = netdev_priv(ndev);
> > +   const char *name;
> > +
> > +   name = dev_name(&rdev->priv->pdev->dev);
> > +   ppid->id_len = min_t(size_t, strlen(name), sizeof(ppid->id_len));
> > +   memcpy(ppid->id, name, ppid->id_len);
>
> Please recheck here sizeof(ppid->id_len) or sizeof(ppid->id) it currently uses sizeof(ppid->id_len),
> which is just 1 byte, so id_len will always be limited to 1.
>
Thank you! Very good catch. I will fix it and send an update.

Best Regards,

Michael

> > +
> > +   return 0;
> > +}
> > +
>
>
> Thanks,
> Alok
________________________________

Renesas Electronics Europe GmbH
Registered Office: Arcadiastrasse 10
DE-40472 Duesseldorf
Commercial Registry: Duesseldorf, HRB 3708
Managing Director: Carsten Jauch
VAT-No.: DE 14978647
Tax-ID-No: 105/5839/1793

Legal Disclaimer: This e-mail communication (and any attachment/s) is confidential and contains proprietary information, some or all of which may be legally privileged. It is intended solely for the use of the individual or entity to which it is addressed. Access to this email by anyone else is unauthorized. If you are not the intended recipient, any disclosure, copying, distribution or any action taken or omitted to be taken in reliance on it, is prohibited and may be unlawful.