Reference: xorg/app/xauth Update DISPLAY parsing to work with new launchd paths in Yosemite

Alan Coopersmith alan.coopersmith at oracle.com
Fri Jan 2 09:45:39 PST 2015


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 <alan.coopersmith at oracle.com>

-- 
	-Alan Coopersmith-              alan.coopersmith at oracle.com
	 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


More information about the xorg-devel mailing list