netdev - Re: [PATCH ethtool 1/2] sfpid: Fix JSON output of SFP diagnostics

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 for Android: free password hash cracker in your pocket

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

Message-ID: <aPjvsU-YEwdIyYha@PC-LX-JohEi>
Date: Wed, 22 Oct 2025 16:52:33 +0200
From: Johannes Eigner <johannes.eigner@...berle.de>
To: Danielle Ratson <danieller@...dia.com>
Cc: "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
	Michal Kubecek <mkubecek@...e.cz>,
	Stephan Wurm <stephan.wurm@...berle.de>
Subject: Re: [PATCH ethtool 1/2] sfpid: Fix JSON output of SFP diagnostics

Am Wed, Oct 22, 2025 at 12:55:45PM +0000 schrieb Danielle Ratson:
> > -----Original Message-----
> > From: Johannes Eigner <johannes.eigner@...berle.de>
> > Sent: Tuesday, 21 October 2025 17:00
> > To: netdev@...r.kernel.org
> > Cc: Michal Kubecek <mkubecek@...e.cz>; Danielle Ratson
> > <danieller@...dia.com>; Stephan Wurm <stephan.wurm@...berle.de>;
> > Johannes Eigner <johannes.eigner@...berle.de>
> > Subject: [PATCH ethtool 1/2] sfpid: Fix JSON output of SFP diagnostics
> > 
> > Close and delete JSON object only after output of SFP diagnostics so
> > that it is also JSON formatted. If the JSON object is deleted too early,
> > some of the output will not be JSON formatted, resulting in mixed output
> > formats.
> > 
> > Signed-off-by: Johannes Eigner <johannes.eigner@...berle.de>
> 
> Hi Johannes,
> 
> Please add a fixes tag.
> 

Will update patches with fixes tags.

> > ---
> >  sfpid.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/sfpid.c b/sfpid.c
> > index 62acb4f..5216ce3 100644
> > --- a/sfpid.c
> > +++ b/sfpid.c
> > @@ -520,8 +520,6 @@ int sff8079_show_all_nl(struct cmd_context *ctx)
> >  	new_json_obj(ctx->json);
> >  	open_json_object(NULL);
> >  	sff8079_show_all_common(buf);
> > -	close_json_object();
> > -	delete_json_obj();
> > 
> >  	/* Finish if A2h page is not present */
> >  	if (!(buf[92] & (1 << 6)))
> > @@ -537,6 +535,8 @@ int sff8079_show_all_nl(struct cmd_context *ctx)
> > 
> >  	sff8472_show_all(buf);
> >  out:
> > +	close_json_object();
> > +	delete_json_obj();
> 
> If sff8079_get_eeprom_page() fails, then close_json_object() and delete_json_object() lines will be called although the object was never opened.
> I think those lines should be after the sff8472_show_all(), but outside the out label.
> 

You are right in case the first sff8079_get_eeprom_page() fails,
close_json_object() and delete_json_object() should not be called. But
for the later goto we need to call close_json_object() and
delete_json_object(). Will add second goto mark to resolve this.

> >  	free(buf);
> > 
> >  	return ret;
> > 
> > --
> > 2.43.0
> 

Download attachment "smime.p7s" of type "application/x-pkcs7-signature" (4153 bytes)