[DNX#2006033142000374] [U-Boot-Users] Re: [patch] add support for AMD Alc [...]
DENX Support System
support at denx.de
Fri Mar 31 23:50:15 CEST 2006
inside the automatic U-Boot patch tracking system a new ticket
[DNX#2006033142000374] was created:
> On 9/24/05, Wolfgang Denk <wd at denx.de> wrote:
> > > I've also fixed the mii and usb commands which can dereference a null
> > > pointer when called without arguments.
> > No, this is not possible. The command definition requires always at
> > least one argument. This part rejected.
> I beg to differ - I've seen this problem also. The code in
> main.c:run_command() checks for more than more than a
> maximum but never checks a minimum number of arguments,
> so entering mii<CR> makes cmd_mii get called with argv = 'mii'
> and argc = 1. main.c:parse_line() sets argv to NULL.
> The code in cmd_mii.c assumes there will be a valid argv,
> but never checks argc before using argv. This leads to an
> exception on machines where 0 is not a valid address (like MIPS).
> Hardware, n.:
> The parts of a computer system that can be kicked.
> SF.Net email is sponsored by:
> Tame your development challenges with Apache's Geronimo App Server.
> it for free - -and be entered to win a 42" plasma tv or your very own
> Sony(tm)PSP. Click here to play: http://sourceforge.net/geronimo.php
> U-Boot-Users mailing list
> U-Boot-Users at lists.sourceforge.net
Your U-Boot support team
More information about the U-Boot