kasan: unify report headers
authorAndrey Konovalov <andreyknvl@google.com>
Wed, 3 May 2017 21:56:28 +0000 (14:56 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Wed, 3 May 2017 22:52:12 +0000 (15:52 -0700)
Unify KASAN report header format for different kinds of bad memory
accesses.  Makes the code simpler.

Link: http://lkml.kernel.org/r/20170302134851.101218-3-andreyknvl@google.com
Signed-off-by: Andrey Konovalov <andreyknvl@google.com>
Acked-by: Dmitry Vyukov <dvyukov@google.com>
Cc: Andrey Ryabinin <aryabinin@virtuozzo.com>
Cc: Alexander Potapenko <glider@google.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
mm/kasan/report.c

index c2bc08b1b5e0a9acd45eb52e7622c662b4fb8d7c..d6b6ec77c56a43cae5785dd2c04d12b032c1f8db 100644 (file)
@@ -121,16 +121,22 @@ const char *get_wild_bug_type(struct kasan_access_info *info)
        return bug_type;
 }
 
+static const char *get_bug_type(struct kasan_access_info *info)
+{
+       if (addr_has_shadow(info))
+               return get_shadow_bug_type(info);
+       return get_wild_bug_type(info);
+}
+
 static void print_error_description(struct kasan_access_info *info)
 {
-       const char *bug_type = get_shadow_bug_type(info);
+       const char *bug_type = get_bug_type(info);
 
        pr_err("BUG: KASAN: %s in %pS at addr %p\n",
-               bug_type, (void *)info->ip,
-               info->access_addr);
+               bug_type, (void *)info->ip, info->access_addr);
        pr_err("%s of size %zu by task %s/%d\n",
-               info->is_write ? "Write" : "Read",
-               info->access_size, current->comm, task_pid_nr(current));
+               info->is_write ? "Write" : "Read", info->access_size,
+               current->comm, task_pid_nr(current));
 }
 
 static inline bool kernel_or_module_addr(const void *addr)
@@ -297,17 +303,11 @@ static void kasan_report_error(struct kasan_access_info *info)
 
        kasan_start_report(&flags);
 
+       print_error_description(info);
+
        if (!addr_has_shadow(info)) {
-               const char *bug_type = get_wild_bug_type(info);
-               pr_err("BUG: KASAN: %s on address %p\n",
-                       bug_type, info->access_addr);
-               pr_err("%s of size %zu by task %s/%d\n",
-                       info->is_write ? "Write" : "Read",
-                       info->access_size, current->comm,
-                       task_pid_nr(current));
                dump_stack();
        } else {
-               print_error_description(info);
                print_address_description(info);
                print_shadow_for_address(info->first_bad_addr);
        }