Hi Stephen, > -----Original Message----- > From: Stephen Hemminger <[email protected]> > Sent: Wednesday, March 22, 2023 11:00 PM > To: Bing Zhao <[email protected]> > Cc: Slava Ovsiienko <[email protected]>; Matan Azrad > <[email protected]>; [email protected]; Raslan Darawsheh > <[email protected]>; [email protected] > Subject: Re: [PATCH v2] common/mlx5: fix the sysfs port name translation > > External email: Use caution opening links or attachments > > > On Wed, 22 Mar 2023 13:34:12 +0200 > Bing Zhao <[email protected]> wrote: > > > + tail_nl = strchr(port_name, '\r'); > > + if (tail_nl) > > + *tail_nl = '\0'; > > This is a file provided by Linux kernel, there is no way it could have a CR > character.
Correct, thanks for pointing this. I was overanxious. I will remove this as well.
