[Sis-ams] validation of Meta-AMS PDUs

[Ray, Timothy J. (GSFC-583.0)]

Perhaps this initial email should have said "validation would probably
be much simpler" rather than "validation would be much simpler".  

 

I'm having a go at performing the validation using the one-byte
string-length prefix to see how much it helps.  I'll report back later
today...

 

Tim

 

________________________________

From: sis-ams-bounces at mailman.ccsds.org
[mailto:sis-ams-bounces at mailman.ccsds.org] On Behalf Of Ray, Timothy J.
(GSFC-583.0)
Sent: Wednesday, June 25, 2008 11:04 AM
To: sis-ams at mailman.ccsds.org
Subject: [Sis-ams] validation of Meta-AMS PDUs

 

Dear WG Members,

 

In my implementation, I am trying to thoroughly validate each incoming
Meta-AMS protocol message (MPDU).  The primary goal is to avoid program
crashes that can occur when an attempt is made to access an array
element beyond the end of the array's allocated memory.  For example, if
the supplementary data includes a field that is supposed to be a
null-terminated character string, but the null-terminator is missing,
avoid any attempts to access past the end of the entire array allocated
to hold the raw MPDU bytes.

 

I'm finding it quite complex to validate MPDUs whose supplementary data
includes null-terminated strings (especially if the strings are buried
within arrays of structures containing sub-structures).  Validation
would be much simpler if, instead of adding a one-byte null terminator
at the end of each string, we added a one-byte length field at the
beginning of each string.  (It would also be possible to add the
one-byte length field and keep the null terminator, although that seems
redundant).

 

Any thoughts/comments?

 

Tim

 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mailman.ccsds.org/pipermail/sis-ams/attachments/20080625/4c331ea7/attachment.htm