On Wed, Dec 06, 2017 at 02:45:15PM -0800, Song Liu wrote:
> @@ -8537,7 +8620,7 @@ static int perf_event_set_filter(struct perf_event 
> *event, void __user *arg)
>       char *filter_str;
>       int ret = -EINVAL;
>  
> -     if ((event->attr.type != PERF_TYPE_TRACEPOINT ||
> +     if ((!perf_event_is_tracing(event) ||
>            !IS_ENABLED(CONFIG_EVENT_TRACING)) &&
>           !has_addr_filter(event))
>               return -EINVAL;

You actually missed an instance later in this same function... fixing
that.

Reply via email to