Julia Lawall schrieb: > From: Julia Lawall <julia@xxxxxxx> > > The size argument to zalloc should be the size of desired structure, > not the pointer to it. > > The semantic patch that makes this change is as follows: > (http://coccinelle.lip6.fr/) > > // <smpl> > @expression@ > expression *x; > @@ > > x = > <+... > -sizeof(x) > +sizeof(*x) > ...+>// </smpl> > > Signed-off-by: Julia Lawall <julia@xxxxxxx> > > --- > tools/perf/util/parse-events.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff -u -p a/tools/perf/util/parse-events.c b/tools/perf/util/parse-events.c > --- a/tools/perf/util/parse-events.c > +++ b/tools/perf/util/parse-events.c > @@ -197,7 +197,7 @@ struct tracepoint_path *tracepoint_id_to > if (id == config) { > closedir(evt_dir); > closedir(sys_dir); > - path = zalloc(sizeof(path)); > + path = zalloc(sizeof(*path)); > path->system = malloc(MAX_EVENT_LENGTH); > if (!path->system) { > free(path); and zalloc() can not fail ? re, wh -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
