[avrdude-dev] [bug #5713] Memory corruption with STK500 and small memory sizes.

[nobody]

=================== BUG #5713: LATEST MODIFICATIONS ==================
http://savannah.nongnu.org/bugs/?func=detailbug&bug_id=5713&group_id=3904

Changes by: Eric Weddington <address@hidden>
Date: Mon 10/13/2003 at 11:37 (America/Denver)

            What     | Removed                   | Added
---------------------------------------------------------------------------
         Assigned to | None                      | arcanum


------------------ Additional Follow-up Comments ----------------------------
Does anyone else want to give this patch a workout before committing? - Eric



=================== BUG #5713: FULL BUG SNAPSHOT ===================


Submitted by: bsomervi                Project: AVR Downloader/UploaDEr      
Submitted on: Fri 10/03/2003 at 07:15
Category:  None                       Severity:  5 - Major                  
Bug Group:  None                      Resolution:  None                     
Assigned to:  arcanum                 Status:  Open                         

Summary:  Memory corruption with STK500 and small memory sizes.

Original Submission:  The stk500.c fn's use paged mode on small memory devices 
e.g. eeprom on smaller devices. This causes memory corruption as paged read 
reads 256/128 bytes in mem->buf which could be as small as 64 bytes.



This can be fixed by reducing the page size to the actual size of the 
read/write (stk500 handles internals itself).



I have a hack that proves this approach works without problems, unfortunately 
not suitable for a patch as it's too messy!

Follow-up Comments
*******************

-------------------------------------------------------
Date: Mon 10/13/2003 at 11:37       By: arcanum
Does anyone else want to give this patch a workout before committing? - Eric

-------------------------------------------------------
Date: Mon 10/06/2003 at 14:45       By: arcanum
This patch works for me for Windows 2000, STK500, ATmega128, writing and 
verifying. Tried it with paged reading on the eeprom on the mega128 (with a 
faked config file) with a small page size (8) and it seemed to work fine.

- Eric

-------------------------------------------------------
Date: Fri 10/03/2003 at 09:06       By: bsomervi
I cleaned up the changes, attached is a patch.


CC list is empty


File Attachments
****************

-------------------------------------------------------
Date: Fri 10/03/2003 at 09:06  Name: bug-5713.patch.bz2  Size: 1KB   By: 
bsomervi
Patch file.
http://savannah.nongnu.org/bugs/download.php?group_id=3904&amp;bug_id=5713&amp;bug_file_id=699


For detailed info, follow this link:
http://savannah.nongnu.org/bugs/?func=detailbug&bug_id=5713&group_id=3904

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/