On Fri, Nov 07, 2014 at 08:03:01AM +0100, k...@shike2.com wrote:
> 
> > - Rename volatile bool q = false; to "isrunning".
> 
> is.* is a reserved name space, it have not to be used in external scope.
> 
> 
> > - Use the safer strlcpy or snprintf instead of strcpy().
> 
> I disagree, check the size before of calling strcpy. If you want to
> avoid security risk you also have to check the output of strlcpy
> to detect truncations, so you don't win anything. In both cases
> you have to add a comparision, so it is better to use strcpy that
> is standard.

Good luck with that.

Reply via email to