Menu

#107 writemem,readmem

0.17
closed
nobody
UrJTAG (101)
5
2018-11-19
2009-11-26
brag
No

In bus/writemem.c in big endian mode word overflow
in bus/writemem, bus/readmem.c the string fopen(params[3],"r"); causes data coruption becaus of translation.
must be fopen(params[3],"rb");
there are files in attachment corrected by me.
thanks.
Igor.

Discussion

  • brag

    brag - 2009-11-26
     
  • Geert Stappers

    Geert Stappers - 2018-11-19
    • status: open --> closed
    • Group: --> 0.17
     

    Last edit: Geert Stappers 2018-11-19
  • Geert Stappers

    Geert Stappers - 2018-11-19

    The patch didn't apply cleanly

    Would you like to replace the existing file:
      Path:     ./bus/readmem.c
      Size:     3715 bytes (4 KiB)
      Modified: 2016-10-30 17:48:41
    with the file from archive:
      Path:     bus/readmem.c
      Size:     2489 bytes (3 KiB)
      Modified: 2009-11-26 23:32:35
    ? (Y)es / (N)o / (A)lways / (S)kip all / A(u)to rename all / (Q)uit? N
    
    Would you like to replace the existing file:
      Path:     ./bus/writemem.c
      Size:     4166 bytes (5 KiB)
      Modified: 2016-10-30 17:48:41
    with the file from archive:
      Path:     bus/writemem.c
      Size:     2463 bytes (3 KiB)
      Modified: 2009-11-26 23:44:22
    ? (Y)es / (N)o / (A)lways / (S)kip all / A(u)to rename all / (Q)uit? N
    
     
  • Geert Stappers

    Geert Stappers - 2018-11-19

    Feel free to resubmit. (most likely there will be a quicker response )

     

    Last edit: Geert Stappers 2018-11-19
  • Geert Stappers

    Geert Stappers - 2018-11-19

    D

     

Log in to post a comment.