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

Jeremy Huddleston Sequoia jeremyhu at apple.com
Fri Jan 2 11:38:05 PST 2015


Sorry about that.

Reviewed-by: Jeremy Huddleston Sequoia <jeremyhu at apple.com>

> On Jan 2, 2015, at 09:45, Alan Coopersmith <alan.coopersmith at oracle.com> 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 <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