On 2015-06-04 15:41, Yousong Zhou wrote: > - Return early on calloc() failure. > - Correct comment text for __calloc_a(). > > Signed-off-by: Yousong Zhou <yszhou4t...@gmail.com> > --- > utils.c | 4 ++++ > utils.h | 6 +++--- > 2 files changed, 7 insertions(+), 3 deletions(-) > > diff --git a/utils.c b/utils.c > index 8fd19f4..627b0f6 100644 > --- a/utils.c > +++ b/utils.c > @@ -43,6 +43,10 @@ void *__calloc_a(size_t len, ...) > va_end(ap1); > > ptr = calloc(1, alloc_len); > + if (!ptr) { > + va_end(ap); > + return NULL; > + } What's the point? The return value without this check will be NULL anyway, and optimizing a rare error case does not seem useful to me.
- Felix _______________________________________________ openwrt-devel mailing list openwrt-devel@lists.openwrt.org https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel