Sorry about that. Reviewed-by: Jeremy Huddleston Sequoia <[email protected]>
> On Jan 2, 2015, at 09:45, Alan Coopersmith <[email protected]> > wrote: > > On 01/ 2/15 09:42 AM, Jon TURNEY wrote: >> On 31/12/2014 17:47, Alan Coopersmith wrote: >>> On 12/31/14 08:11 AM, Colin Harrison wrote: >>>> Hi, >>>> >>>> In the latest commit to xauth there are two added instances of: >>>> >>>> buf[sizeof(path) - 1] = '\0'; >>>> >>>> shouldn't those be? >>>> path[sizeof(path) - 1] = '\0'; >>> >>> Looks like it, for those OS'es without a good enough libc to have strlcpy. >> >> Patch attached. > > Reviewed-by: Alan Coopersmith <[email protected]> > > -- > -Alan Coopersmith- [email protected] > Oracle Solaris Engineering - http://blogs.oracle.com/alanc _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
