On 21.11.2015 20:16, Michael Gilbert wrote:
> On Sat, Nov 21, 2015 at 11:47 AM, Svante Signell wrote:
>> Signed-off by: svante.signell(a)gmail.com
>>
>> Hello,
>>
>> Submitting this patch directly to wine-patches on request from Jens
>> Ryer.
>>
>> BTW: There are still some parts of the code using PATH_MAX, but does
>> not seem to be triggered with the debian builds. Is it of interest to
>> remove also the remaining entries?
>
> The PATH_MAX in libs/wine/loader.c is in an #ifdef __sun block, so
> it's not very relevant. There are two other instances which would
> probably be nice to fix.
winepulse.drv makes perfect sense, since this buffer is used with Win32
API, and pa_proplist_sets() dynamically allocates property value, so it
could be of any length.
The rest:
- loader.c is fine I think;
- mountmgr.sys/diskarb.c and dbghelp/macho_module.c are macos-only,
so maybe it's okay too if PATH_MAX defined specifically for macos.
Ken, could you please comment?
>
> Best wishes,
> Mike
>
>