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