STK500V2 Protocol Bug in Mega 2560 Bootloader [imported]

[cmaglie]

This is Issue 505 moved from a Google Code project.
Added by 2011-03-15T03:50:52.000Z by [email protected].
Please review that bug for more context and additional comments, but update this bug.

Original labels: Type-Defect, Priority-Medium, Component-stk500v2

Original description

The STK500V2 bootloader is comparing the seqNum to 1 or the current sequence (IE: Requiring the sequence to be 1 or match seqNum before continuing).

The correct behavior is for the STK500V2 to accept the PC's sequence number, and echo it back for the reply message.

Here is the fix for the stk500boot.c file:

                case ST_GET_SEQ_NUM:
                    seqNum          =   c;    
                    msgParseState   =   ST_MSG_SIZE_1;
                    checksum        ^=  c;
                    break;

This is on line 600 of the local copy of the source I have.

[cmaglie]

Please look at the pullrequest here #1183

The updated hex bootloader is available here:

https://github.com/cmaglie/Arduino/blob/mega2560-bootloader/hardware/arduino/bootloaders/stk500v2/stk500boot_v2_mega2560.hex

Please check it out and report here #1183 success or failures.
Thank you!

C