ravthan / all-eyes

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

Module aemsg.c, function aemsg.c uses strcpy #109

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Module aemsg.c, function aemsg.c uses strcpy 8 times without ever checking the 
length of the strings – recommend replace strcpy with strncpy and limit to 
the length of the string for each portion of the message. There are comments in 
the file indicating that the AE team is aware of this issue

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

GoogleCodeExporter commented 8 years ago
Yes we are aware of this issue as stated in the issue.  Will fix it,
ravi.

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

GoogleCodeExporter commented 8 years ago
Changed all the strcpy to strncpy with proper sizes.
Fixed.

r976

Original comment by ravt...@gmail.com on 25 Nov 2012 at 9:04

GoogleCodeExporter commented 8 years ago
Verified with unit testing

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