Hello,

> if (groups) free(groups);

I think these checks are redundant, because according to free(3):
-- If ptr is NULL, no action occurs.


On 29 апр. 2014 г., at 3:00, Dirkjan Bussink <d.buss...@gmail.com> wrote:

> Hi all,
> 
> When building HAProxy using the Clang Static Analyzer, it found a few cases 
> of invalid memory usage and leaks. I’ve attached a patch to fix these cases.
> 
> — 
> Regards,
> 
> Dirkjan Bussink
> 
> <0001-Fix-a-few-memory-usage-errors.patch>


Reply via email to