Re: [OpenSIPS-Devel] [opensips] F_MALLOC: suppress warnings for free(NULL) (17d639b)

2015-06-15 Thread Liviu Chircu
the libc free() (man 3 free) allows free'ing NULL pointers. I think the quality of the codebase is improved if we stick to this standard. Your suggestion is excellent. We definitely have to remove all `if (buf)` tests! --- Reply to this email directly or view it on GitHub: https://github.com/Ope

Re: [OpenSIPS-Devel] [opensips] F_MALLOC: suppress warnings for free(NULL) (17d639b)

2015-06-15 Thread Walter Doekes
Shouldn't you fix the caller that pkg_free's without checking for NULL instead? If you "allow" it, then you might as well drop all the `if(buf)` before every `pkg_free(buf)`. Allowing both makes the code inconsistent. Right? --- Reply to this email directly or view it on GitHub: https://github.