diff options
author | Steven Rostedt <srostedt@redhat.com> | 2009-02-19 18:41:27 (GMT) |
---|---|---|
committer | Steven Rostedt <srostedt@redhat.com> | 2009-02-20 19:30:20 (GMT) |
commit | 4377245aa93b65b6597e4b7bb460fb9abc48b56b (patch) | |
tree | 00eae2cbdd12e855bbde2f6cd6d1a6e6fd659a11 /kernel/trace/ftrace.c | |
parent | 90c7ac49aa819feb9433b5310089fca6399881c0 (diff) | |
download | linux-fsl-qoriq-4377245aa93b65b6597e4b7bb460fb9abc48b56b.tar.xz |
ftrace: break out modify loop immediately on detection of error
Impact: added precaution on failure detection
Break out of the modifying loop as soon as a failure is detected.
This is just an added precaution found by code review and was not
found by any bug chasing.
Signed-off-by: Steven Rostedt <srostedt@redhat.com>
Diffstat (limited to 'kernel/trace/ftrace.c')
-rw-r--r-- | kernel/trace/ftrace.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c index 72316d9..11ad796 100644 --- a/kernel/trace/ftrace.c +++ b/kernel/trace/ftrace.c @@ -561,8 +561,11 @@ static void ftrace_replace_code(int enable) if ((system_state == SYSTEM_BOOTING) || !core_kernel_text(rec->ip)) { ftrace_free_rec(rec); - } else + } else { ftrace_bug(failed, rec->ip); + /* Stop processing */ + return; + } } } } |