[prev in list] [next in list] [prev in thread] [next in thread] 

List:       freedesktop-xorg-devel
Subject:    Re: Reference: xorg/app/xauth Update DISPLAY parsing to work with new launchd paths in Yosemite
From:       Alan Coopersmith <alan.coopersmith () oracle ! com>
Date:       2014-12-31 17:47:57
Message-ID: 54A436CD.4020001 () oracle ! com
[Download RAW message or body]

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.

-- 
	-Alan Coopersmith-              alan.coopersmith@oracle.com
	 Oracle Solaris Engineering - http://blogs.oracle.com/alanc
_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
[prev in list] [next in list] [prev in thread] [next in thread]