On 12:34 Mon 12 Aug     , Willy Tarreau wrote:
> Hi Apollon,
> 
> Unfortunately, this patch introduces a memory leak. Since the path
> variable is just a temporary one, better use alloca() instead in
> order to dynamically allocate on the stack.

Ooops, completely forgot to free(). Never submit patches directly after 
vacation :-(

> Anyway I'd prefer something simpler : let's define PATH_MAX in compat.h
> if it is not defined.

Yes, that's probably a cleaner solution.

> Thanks,
> Willy

Thanks,
Apollon


Reply via email to