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:


  Message sent via/by Gna!

Freeciv-dev mailing list

Reply via email to