search

ravthan / all-eyes

Automatically exported from code.google.com/p/all-eyes
0 stars 0 forks source link

Modify chkAeMsgIntegrity to handle MSG_TRAILER as it handles AE_MSG_HEADER #107

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
Module aemsg.c, function chkAeMsgIntegrity seems to do a good job of validating 
the msg string.  AE_MSG_HEADER is handled in an abstract manner, allowing for 
easy modification of the header, while the check for the trailer is hard coded 
to the two specific characters defined as the trailer – not really an error 
but more an issue of coding style/re-usability/maintenance

Original issue reported on code.google.com by prhamm...@gmail.com on 24 Nov 2012 at 7:51

GoogleCodeExporter commented 8 years ago 
Good suggestion for better coding.
Implemented it.

r973

Original comment by ravt...@gmail.com on 25 Nov 2012 at 6:08

GoogleCodeExporter commented 8 years ago 
Verified with unit testing

Original comment by tbrt....@gmail.com on 9 Dec 2012 at 9:24