linux-kernel - Re: [RFC][PATCH 07/23 v4] kprobes/tracing: Use trace_seq_has

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 14 Nov 2014 23:51:58 +0900
From:	Masami Hiramatsu <masami.hiramatsu.pt@...achi.com>
To:	Steven Rostedt <rostedt@...dmis.org>
Cc:	linux-kernel@...r.kernel.org, Ingo Molnar <mingo@...nel.org>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Jiri Kosina <jkosina@...e.cz>, Petr Mladek <pmladek@...e.cz>
Subject: Re: [RFC][PATCH 07/23 v4] kprobes/tracing: Use trace_seq_has_overflowed()
 for overflow checks

(2014/11/14 10:12), Steven Rostedt wrote:
> From: "Steven Rostedt (Red Hat)" <rostedt@...dmis.org>
> 
> Instead of checking the return value of trace_seq_printf() and friends
> for overflowing of the buffer, use the trace_seq_has_overflowed() helper
> function.
> 
> This cleans up the code quite a bit and also takes us a step closer to
> changing the return values of trace_seq_printf() and friends to void.

Looks good to me :)

Acked-by: Masami Hiramatsu <masami.hiramatsu.pt@...achi.com>

Thank you,

> 
> Cc: Masami Hiramatsu <masami.hiramatsu.pt@...achi.com>
> Signed-off-by: Steven Rostedt <rostedt@...dmis.org>
> ---
>  kernel/trace/trace_kprobe.c | 42 ++++++++++++++++--------------------------
>  1 file changed, 16 insertions(+), 26 deletions(-)
> 
> diff --git a/kernel/trace/trace_kprobe.c b/kernel/trace/trace_kprobe.c
> index 282f6e4e5539..be14b2f091c8 100644
> --- a/kernel/trace/trace_kprobe.c
> +++ b/kernel/trace/trace_kprobe.c
> @@ -1024,27 +1024,22 @@ print_kprobe_event(struct trace_iterator *iter, int flags,
>  	field = (struct kprobe_trace_entry_head *)iter->ent;
>  	tp = container_of(event, struct trace_probe, call.event);
>  
> -	if (!trace_seq_printf(s, "%s: (", ftrace_event_name(&tp->call)))
> -		goto partial;
> +	trace_seq_printf(s, "%s: (", ftrace_event_name(&tp->call));
>  
>  	if (!seq_print_ip_sym(s, field->ip, flags | TRACE_ITER_SYM_OFFSET))
> -		goto partial;
> +		goto out;
>  
> -	if (!trace_seq_puts(s, ")"))
> -		goto partial;
> +	trace_seq_puts(s, ")");
>  
>  	data = (u8 *)&field[1];
>  	for (i = 0; i < tp->nr_args; i++)
>  		if (!tp->args[i].type->print(s, tp->args[i].name,
>  					     data + tp->args[i].offset, field))
> -			goto partial;
> -
> -	if (!trace_seq_puts(s, "\n"))
> -		goto partial;
> +			goto out;
>  
> -	return TRACE_TYPE_HANDLED;
> -partial:
> -	return TRACE_TYPE_PARTIAL_LINE;
> +	trace_seq_puts(s, "\n");
> + out:
> +	return trace_handle_return(s);
>  }
>  
>  static enum print_line_t
> @@ -1060,33 +1055,28 @@ print_kretprobe_event(struct trace_iterator *iter, int flags,
>  	field = (struct kretprobe_trace_entry_head *)iter->ent;
>  	tp = container_of(event, struct trace_probe, call.event);
>  
> -	if (!trace_seq_printf(s, "%s: (", ftrace_event_name(&tp->call)))
> -		goto partial;
> +	trace_seq_printf(s, "%s: (", ftrace_event_name(&tp->call));
>  
>  	if (!seq_print_ip_sym(s, field->ret_ip, flags | TRACE_ITER_SYM_OFFSET))
> -		goto partial;
> +		goto out;
>  
> -	if (!trace_seq_puts(s, " <- "))
> -		goto partial;
> +	trace_seq_puts(s, " <- ");
>  
>  	if (!seq_print_ip_sym(s, field->func, flags & ~TRACE_ITER_SYM_OFFSET))
> -		goto partial;
> +		goto out;
>  
> -	if (!trace_seq_puts(s, ")"))
> -		goto partial;
> +	trace_seq_puts(s, ")");
>  
>  	data = (u8 *)&field[1];
>  	for (i = 0; i < tp->nr_args; i++)
>  		if (!tp->args[i].type->print(s, tp->args[i].name,
>  					     data + tp->args[i].offset, field))
> -			goto partial;
> +			goto out;
>  
> -	if (!trace_seq_puts(s, "\n"))
> -		goto partial;
> +	trace_seq_puts(s, "\n");
>  
> -	return TRACE_TYPE_HANDLED;
> -partial:
> -	return TRACE_TYPE_PARTIAL_LINE;
> + out:
> +	return trace_handle_return(s);
>  }
>  
>  


-- 
Masami HIRAMATSU
Software Platform Research Dept. Linux Technology Research Center
Hitachi, Ltd., Yokohama Research Laboratory
E-mail: masami.hiramatsu.pt@...achi.com


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