Re: [PATCH 2/5] tracing: add tracer dependent options to optionsdirectory

From: Andrew Morton
Date: Fri Feb 27 2009 - 00:51:22 EST


On Fri, 27 Feb 2009 00:34:12 -0500 Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:

> + topts = kzalloc(sizeof(*topts) * (cnt + 1), GFP_KERNEL);

nit: kcalloc() checks for multiplicative overflow, whereas the above
does not. It sets a good example ;)

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/