Re: [PATCH 0/1] MR6152: msvcrt, winex11, mountmgr: fixed memleaks

[Piotr Caban (＠piotr)]

Piotr Caban (@piotr) commented about dlls/msvcrt/dir.c:
> if (size < 4)
>    {
> - if (alloced)
> -
>  free(buffer);

There's no leak in this function. `size` is set to `MAX_PATH` when `alloced` is set to TRUE. A quick testing shows that the function may be improved though (e.g. we shouldn't ignore `size` parameter when buffer gets allocated). How did you spot this? Is there any compiler that prints a warning on this code?