Messages in this thread | | | Date | Fri, 07 Sep 2012 00:22:45 +0200 | From | Toralf Förster <> | Subject | Re: does gcc gives a false warning in kernel/trace/trace_events_filter.c ? |
| |
On 09/06/2012 07:31 PM, Steven Rostedt wrote: > On Thu, 2012-09-06 at 18:35 +0200, Toralf Förster wrote: > >> I filed a bug report >> >> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54495 >> and got this answer : >> >> --- Comment #1 from Hans-Peter Nilsson <hp at gcc dot gnu.org> 2012-09-05 22:14:00 UTC --- >> But if the call to ftrace_function_filter_re sets re_cnt to 0, then ret indeed >> will be used uninitialized AFAICT. What am I missing? >> > > That I think we are looking at two different code bases ;-) > > I've been looking at what's been queued for 3.7 and not what's in > mainline. If you look at tip/master, or even linux-next, you'll find: > > commit 92d8d4a8b0f "tracing/filter: Add missing initialization" > > Which does: > > static int __ftrace_function_set_filter(int filter, char *buf, int len, > struct function_filter_data *data) > { > - int i, re_cnt, ret; > + int i, re_cnt, ret = -EINVAL; > int *reset; > char **re; > > > Thus, you were correct. This could have been marked urgent, but as it > isn't that big of a deal I just queued it for the next merge window. > > -- Steve > > > ah - thx :-)
-- MfG/Sincerely Toralf Förster pgp finger print: 7B1A 07F4 EC82 0F90 D4C2 8936 872A E508 7DB6 9DA3 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |