Fred's unofficial TODO list from his favorite nitpicker

Official FreeEMS vanilla firmware development, the heart and soul of the system!
User avatar
Fred
Moderator
Posts: 15431
Joined: Tue Jan 15, 2008 2:31 pm
Location: Home sweet home!
Contact:

Re: Fred's unofficial TODO list from his favorite nitpicker

Post by Fred »

Hold on a second, such a packet should never have made it through, I need more info! Is that length the actual length (not possible) or the reported length (it would be more likely for me to be killed by lightning in the next 5 minutes than this not be true). You should really just reject it and spit out "length field mismatch with actual length" error or something equivalent. It should be easy to fix, one moment.
DIYEFI.org - where Open Source means Open Source, and Free means Freedom
FreeEMS.org - the open source engine management system
FreeEMS dev diary and its comments thread and my turbo truck!
n00bs, do NOT PM or email tech questions! Use the forum!
The ever growing list of FreeEMS success stories!
dandruczyk
LQFP112 - Up with the play
Posts: 100
Joined: Sun Apr 06, 2008 6:30 pm

Re: Fred's unofficial TODO list from his favorite nitpicker

Post by dandruczyk »

the packet length is 101 bytes as it said in the debug message, but the header WITHIN the packet says the payload is way larger than it should be....
User avatar
Fred
Moderator
Posts: 15431
Joined: Tue Jan 15, 2008 2:31 pm
Location: Home sweet home!
Contact:

Re: Fred's unofficial TODO list from his favorite nitpicker

Post by Fred »

Fixed!
DIYEFI.org - where Open Source means Open Source, and Free means Freedom
FreeEMS.org - the open source engine management system
FreeEMS dev diary and its comments thread and my turbo truck!
n00bs, do NOT PM or email tech questions! Use the forum!
The ever growing list of FreeEMS success stories!
Post Reply