[PATCH] Remove strcpy() from most Team-controlled code.

Jeremy Allison jra at samba.org
Mon Mar 21 17:35:21 UTC 2016


On Sat, Mar 19, 2016 at 11:21:47AM +1100, Martin Schwenke wrote:
> 
> Yeah, we'll get there.  Some of the code in the ctdb CLI tool (written
> by me) uses strcpy(3) (perhaps even technically incorrectly) because it
> is simple and obvious. There is a buffer declared, there are a finite
> number of strings declared nearby that can be concatenated into the
> buffer.  It will fit.  You'd have to be trying hard to change
> the code so it actually breaks and overflows the buffer.  More famous
> last words!  I'll take another look at my patch for it on Monday and
> probably post it... and then Amitay will throw it all away when he
> finally gets time to apply the final tweaks to his replacement...  :-)
> 
> peace & happiness,
> martin

LGTM. Pushed ! Thanks.



More information about the samba-technical mailing list