From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jonathan Larmour To: Grant Edwards Cc: ecos-discuss@sources.redhat.com Subject: Re: [ECOS] RedBoot: load.c srecord input offset fix Date: Fri, 20 Apr 2001 00:11:00 -0000 Message-id: <3ADFE122.BBBFA800@redhat.com> References: <20010419152033.A29215@visi.com> X-SW-Source: 2001-04/msg00250.html Grant Edwards wrote: > > The "offset" variable in load_srec_image() isn't incremented > properly (assuming its purpose is to keep track of the current > byte offset in the input stream). My version of load.c has > diverged enough that I can't generate a usable patch, so I'll > summarize the changes: [snip] I'm probably being dumb but this doesn't seem right to me either. If it's purely the offset within the I/O stream, then there should be one per getc. If it's the offset of the decoded data bytes, there should be one for every two chars of actual encoded srec data, and the offset shouldn't be incremented for any of the header or checksum. What is the "offset" actually meant to _be_ if neither of those two options? Jifl -- Red Hat, Rustat House, Clifton Road, Cambridge, UK. Tel: +44 (1223) 271062 Maybe this world is another planet's Hell -Aldous Huxley || Opinions==mine