On Wed, 13 Aug 2014 13:48:22 +0200, Jiri Olsa wrote: > On Tue, Aug 12, 2014 at 03:40:35PM +0900, Namhyung Kim wrote: >> + annotate.session = perf_session__new(&file, false, &annotate.tool); >> + if (annotate.session == NULL) >> + return -ENOMEM; > > I know you're just moving code, but perf_session__new could > fail for more reasons than just ENOMEM, which is the most > unlikely case ;-) -1 is probably better option here
Okay, will change as a separate fix later. Thanks, Namhyung -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/