[DNX#2006040142000688] [U-Boot-Users] Re: parse bug in cmd_mii.c
DENX Support System
support at denx.de
Sat Apr 1 00:20:30 CEST 2006
Hello list,
inside the automatic U-Boot patch tracking system a new ticket
[DNX#2006040142000688] was created:
<snip>
> Sorry, it looks like I never sent it in. Here goes:
>
> CHANGELOG
> * entering 'mii' with no arguments causes a null pointer reference.
> Check argc is >=2 before dereferencing argv[1]. Print the usage message
> if
> not.
> Patch by Andrew Dyer <amdyer at gmail.com>, Sat Mar 11 21:42:12 CST 2006
>
> Signed-off-by: Andrew Dyer <amdyer at gmail.com>
>
> Index: cmd_mii.c
> ===================================================================
> RCS file: /home/cvsroot/Projects/u-boot/common/cmd_mii.c,v
> retrieving revision 1.1.1.5
> retrieving revision 1.8
> diff -u -r1.1.1.5 -r1.8
> --- cmd_mii.c 7 Feb 2006 09:51:23 -0000 1.1.1.5
> +++ cmd_mii.c 12 Mar 2006 03:33:56 -0000 1.8
> @@ -61,6 +61,11 @@
> mii_init ();
> #endif
>
> + if (argc < 2) {
> + printf ("Usage:\n%s\n", cmdtp->usage);
> + return 1;
> + }
> +
> /*
> * We use the last specified parameters, unless new ones are
> * entered.
> @@ -438,6 +443,11 @@
> #ifdef CONFIG_8xx
> mii_init ();
> #endif
> +
> + if (argc < 2) {
> + printf ("Usage:\n%s\n", cmdtp->usage);
> + return 1;
> + }
>
> /*
> * We use the last specified parameters, unless new ones are
>
>
>
>
> On 3/11/06, Wolfgang Denk <wd at denx.de> wrote:
> > In message <c166aa9f0506131006ba8f552 at mail.gmail.com> you wrote:
> > > I noticed that in do_mii() the code is dereferencing argv[1] without
> > > checking what value argc has. This leads to an exception if a command
> > > line like 'mii ' is given where argv[1] is NULL.
> > >
> > > I have a patch, but my cmd_mii.c is carrying around some other patches
> > > (some submitted to the list, some local). Is there a good way to
> > > separate out the patches I want to submit by themselves?
> >
> > Did I miss it, or did you never send a patch for the problem you
> > mentioned?
> >
> >
> > Best regards,
> >
> > Wolfgang Denk
> >
> > --
> > Software Engineering: Embedded and Realtime Systems, Embedded Linux
> > Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
> > "'Tis true, 'tis pity, and pity 'tis 'tis true."
> > - Poloniouius, in Willie the Shake's _Hamlet, Prince of Darkness_
> >
>
>
> --
> Hardware, n.:
> The parts of a computer system that can be kicked.
>
>
> -------------------------------------------------------
> This SF.Net email is sponsored by xPML, a groundbreaking scripting
> language
> that extends applications into web and mobile media. Attend the live
> webcast
> and join the prime developer group breaking into this new coding
> territory!
> http://sel.as-us.falkag.net/sel?cmd=lnk&kid0944&bid$1720&dat1642
> _______________________________________________
> U-Boot-Users mailing list
> U-Boot-Users at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/u-boot-users
</snip>
Your U-Boot support team
More information about the U-Boot
mailing list