Learn how easy it is to sync an existing GitHub or Google Code repo to a SourceForge project! See Demo

Close

#103 Possible buffer overrun in amd_detect.

0.x
closed-accepted
Arnim Läuger
UrJTAG (102)
5
2009-07-17
2009-07-15
MURANAKA Masaki
No

I've not encountered crashes as this issue will not be occured in case bus width is 8.
But this is obvious bug, right?

Discussion

  • Arnim Läuger
    Arnim Läuger
    2009-07-15

    • labels: --> UrJTAG
    • milestone: --> 0.x
    • assigned_to: nobody --> arniml
    • status: open --> open-accepted
     
  • Arnim Läuger
    Arnim Läuger
    2009-07-15

    Applied as r1661, thanks.
    amd_flash.c was inherited as a patch against the original openwince-jtag. It seems to me that the whole file is obsolete in the meantime. Do you have a board/hardware flash that actually relies on the code in amd_flash.c? I'd expect that all chips supported by amd_flash.c are covered by jedec.c/amd.c

     
  • Thanks for apply my patch.
    I agree it's better we merge amd_flash.c and amd.c up. I'll try later.

     
    • status: open-accepted --> closed-accepted