Follow-up Comment #1, bug #18111 (project freeciv):

First change this patch does is:
> -  strcpy(dest, str);
> +  fc_strlcpy(dest, str, strlen(str)+1);

How that makes anything safer? Seems to me that it only obscures the fact
that it's unsafe. Size of strcpy() area is already strlen()+1, and your change
does not give any protection against too small dest or missing ' ' from where
str should end.

I didn├Ąt read patch any further this time.

    _______________________________________________________

Reply to this item at:

  <http://gna.org/bugs/?18111>

_______________________________________________
  Message sent via/by Gna!
  http://gna.org/


_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

Reply via email to