linux-kernel - Re: [PATCH] trace-cmd: Handle __print_hex(__get_dynamic

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 for Android: free password hash cracker in your pocket

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

Message-ID: <87ob6sbq8f.fsf@sejong.aot.lge.com>
Date:	Mon, 14 Oct 2013 10:22:56 +0900
From:	Namhyung Kim <namhyung@...nel.org>
To:	Howard Cochran <hcochran@...mark.com>
Cc:	rostedt@...dmis.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] trace-cmd: Handle __print_hex(__get_dynamic_array(fieldname), len)

Hi Howard,

On Fri, 11 Oct 2013 10:55:49 -0400, Howard Cochran wrote:
> The kernel has a few events with a format similar to this excerpt:
>         field:unsigned int len;     offset:12;      size:4; signed:0;
>         field:__data_loc unsigned char[] data_array;  offset:16;      size:4; signed:0;
> print fmt: "%s", __print_hex(__get_dynamic_array(data_array), REC->len)
>
> trace-cmd could already parse that arg correctly, but print_str_arg()
> was unable to handle the first parameter being a dynamic array. (It
> just printed a "field not found" warning).
>
> Teach print_str_arg's PRINT_HEX case to handle the nested
> PRINT_DYNAMIC_ARRAY correctly. The output now matches the kernel's own
> formatting for this case.

It seems that it needs to be added to libtraceevent also.  Could you
also send a patch against tools/lib/traceevent/event-parse.c in the
Linux kernel source?

Thanks,
Namhyung

>
> Signed-off-by: Howard Cochran <hcochran@...mark.com>
> ---
>  event-parse.c |   24 ++++++++++++++++--------
>  1 file changed, 16 insertions(+), 8 deletions(-)
>
> diff --git a/event-parse.c b/event-parse.c
> index e961553..fdb176e 100644
> --- a/event-parse.c
> +++ b/event-parse.c
> @@ -3550,15 +3550,23 @@ static void print_str_arg(struct trace_seq *s, void *data, int size,
>  		}
>  		break;
>  	case PRINT_HEX:
> -		field = arg->hex.field->field.field;
> -		if (!field) {
> -			str = arg->hex.field->field.name;
> -			field = pevent_find_any_field(event, str);
> -			if (!field)
> -				goto out_warning_field;
> -			arg->hex.field->field.field = field;
> +		if (PRINT_DYNAMIC_ARRAY == arg->hex.field->type) {
> +			unsigned long offset;
> +			offset = pevent_read_number(pevent,
> +				data + arg->hex.field->dynarray.field->offset,
> +				arg->hex.field->dynarray.field->size);
> +			hex = data + (offset & 0xffff);
> +		} else {
> +			field = arg->hex.field->field.field;
> +			if (!field) {
> +				str = arg->hex.field->field.name;
> +				field = pevent_find_any_field(event, str);
> +				if (!field)
> +					goto out_warning_field;
> +				arg->hex.field->field.field = field;
> +			}
> +			hex = data + field->offset;
>  		}
> -		hex = data + field->offset;
>  		len = eval_num_arg(data, size, event, arg->hex.size);
>  		for (i = 0; i < len; i++) {
>  			if (i)
--
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/