From cc4f887d47f83de6cdfd4fc0f93fffc81427cb36 Mon Sep 17 00:00:00 2001 From: Simon Marchi Date: Thu, 8 Nov 2012 15:17:54 -0500 Subject: [PATCH] Check if interrupt action name is not null in statedump Protection against poorly written drivers who give a null name to their irq handlers. Signed-off-by: Simon Marchi Signed-off-by: Mathieu Desnoyers --- instrumentation/events/lttng-module/lttng-statedump.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/instrumentation/events/lttng-module/lttng-statedump.h b/instrumentation/events/lttng-module/lttng-statedump.h index d350cad3..e4c86d65 100644 --- a/instrumentation/events/lttng-module/lttng-statedump.h +++ b/instrumentation/events/lttng-module/lttng-statedump.h @@ -150,12 +150,12 @@ TRACE_EVENT(lttng_statedump_interrupt, TP_STRUCT__entry( __field(unsigned int, irq) __string(name, chip_name) - __string(action, action->name) + __string(action, action->name ? : "") ), TP_fast_assign( tp_assign(irq, irq) tp_strcpy(name, chip_name) - tp_strcpy(action, action->name) + tp_strcpy(action, action->name ? : "") ), TP_printk("") ) -- 2.34.1