[PATCH] drivers/staging/sep: using strlcpy instead of strncpy
Chen Gang
gang.chen at asianux.com
Sun Feb 17 03:53:15 UTC 2013
set '\0' at tail for NUL terminated string, or TP_printk may cause issue.
Signed-off-by: Chen Gang <gang.chen at asianux.com>
---
drivers/staging/sep/sep_trace_events.h | 11 ++++++++---
1 files changed, 8 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/sep/sep_trace_events.h b/drivers/staging/sep/sep_trace_events.h
index 2b053a9..74f4c9a 100644
--- a/drivers/staging/sep/sep_trace_events.h
+++ b/drivers/staging/sep/sep_trace_events.h
@@ -53,6 +53,11 @@
#include <linux/tracepoint.h>
/*
+ * Since use str*cpy in header file, better to include string.h, directly.
+ */
+#include <linux/string.h>
+
+/*
* The TRACE_EVENT macro is broken up into 5 parts.
*
* name: name of the trace point. This is also how to enable the tracepoint.
@@ -97,7 +102,7 @@ TRACE_EVENT(sep_func_start,
),
TP_fast_assign(
- strncpy(__entry->name, name, 20);
+ strlcpy(__entry->name, name, 20);
__entry->branch = branch;
),
@@ -116,7 +121,7 @@ TRACE_EVENT(sep_func_end,
),
TP_fast_assign(
- strncpy(__entry->name, name, 20);
+ strlcpy(__entry->name, name, 20);
__entry->branch = branch;
),
@@ -135,7 +140,7 @@ TRACE_EVENT(sep_misc_event,
),
TP_fast_assign(
- strncpy(__entry->name, name, 20);
+ strlcpy(__entry->name, name, 20);
__entry->branch = branch;
),
--
1.7.7.6
More information about the devel
mailing list