[ECOS] Re[ECOS] dBoot: "fis load -d" allows "go" even if it fails

[ECOS] Re[ECOS] dBoot: "fis load -d" allows "go" even if it fails

[René Nielsen]

Hello,

If you do a
  fis load -d my_image

and my_image is not a valid zip-image, it will not be loaded. That's fine.
But a following "go" will jump to the img->entry_point anyway. 

When loading an image without the -d option and CYGSEM_REDBOOT_FIS_CRC_CHECK
is defined and the checksum-check fails, it will set entry_addr to
NO_MEMORY. I think the same thing should happen when the unzip fails.

Regards
René Schipp von Branitz Nielsen
Vitesse Semiconductors

-- 
View this message in context: http://www.nabble.com/RedBoot%3A-%22fis-load--d%22-allows-%22go%22-even-if-it-fails-tf4888998.html#a13993780
Sent from the Sourceware - ecos-discuss mailing list archive at Nabble.com.


--
Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos
and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss

Re: [ECOS] Re[ECOS] dBoot: "fis load -d" allows "go" even if it fails

[Gary Thomas]

René Nielsen wrote:
> Hello,
> 
> If you do a
>   fis load -d my_image
> 
> and my_image is not a valid zip-image, it will not be loaded. That's fine.
> But a following "go" will jump to the img->entry_point anyway. 
> 
> When loading an image without the -d option and CYGSEM_REDBOOT_FIS_CRC_CHECK
> is defined and the checksum-check fails, it will set entry_addr to
> NO_MEMORY. I think the same thing should happen when the unzip fails.

Send a patch :-)

BTW, I've looked at your other patch and in general I think it's
fine.  It is rather involved, so we need a Copyright assignment
before I can consider it for commitment.
  http://ecos.sourceware.org/assign.html

-- 
------------------------------------------------------------
Gary Thomas                 |  Consulting for the
MLB Associates              |    Embedded world
------------------------------------------------------------

-- 
Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos
and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss

[ECOS] Re: Re[ECOS] dBoot: "fis load -d" allows "go" even if it fails

[René Nielsen]

> Send a patch :-)
> BTW, I've looked at your other patch and in general I think it's
> fine.  

I've merged the patch with the zip-auto-detect patch in the ecos-devel
mailing list (see e.g.
http://www.nabble.com/forum/ViewPost.jtp?post=14042399&framed=y).

> It is rather involved, so we need a Copyright assignment
> before I can consider it for commitment.
>   http://ecos.sourceware.org/assign.html
Done.

/René

-- 
View this message in context: http://www.nabble.com/RedBoot%3A-%22fis-load--d%22-allows-%22go%22-even-if-it-fails-tf4888998.html#a14042405
Sent from the Sourceware - ecos-discuss mailing list archive at Nabble.com.


--
Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos
and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss