diff mbox series

[mm,v2,25/33] kasan: simplify print_report

Message ID	ca95c73bf01ea28e3d9324b170a9a86ff2b82b81.1658189199.git.andreyknvl@google.com (mailing list archive)
State	New
Headers	show Return-Path: <owner-linux-mm@kvack.org> From: andrey.konovalov@linux.dev To: Marco Elver <elver@google.com>, Alexander Potapenko <glider@google.com> Cc: Andrey Konovalov <andreyknvl@gmail.com>, Dmitry Vyukov <dvyukov@google.com>, Andrey Ryabinin <ryabinin.a.a@gmail.com>, kasan-dev@googlegroups.com, Peter Collingbourne <pcc@google.com>, Evgenii Stepanov <eugenis@google.com>, Florian Mayer <fmayer@google.com>, Andrew Morton <akpm@linux-foundation.org>, linux-mm@kvack.org, linux-kernel@vger.kernel.org, Andrey Konovalov <andreyknvl@google.com> Subject: [PATCH mm v2 25/33] kasan: simplify print_report Date: Tue, 19 Jul 2022 02:10:05 +0200 Message-Id: <ca95c73bf01ea28e3d9324b170a9a86ff2b82b81.1658189199.git.andreyknvl@google.com> In-Reply-To: <cover.1658189199.git.andreyknvl@google.com> References: <cover.1658189199.git.andreyknvl@google.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: owner-linux-mm@kvack.org Precedence: bulk
Series	kasan: switch tag-based modes to stack ring from per-object metadata \| expand [mm,v2,00/33] kasan: switch tag-based modes to stack ring from per-object metadata [mm,v2,01/33] kasan: check KASAN_NO_FREE_META in __kasan_metadata_size [mm,v2,02/33] kasan: rename kasan_set__info to kasan_save__info [mm,v2,03/33] kasan: move is_kmalloc check out of save_alloc_info [mm,v2,04/33] kasan: split save_alloc_info implementations [mm,v2,05/33] kasan: drop CONFIG_KASAN_TAGS_IDENTIFY [mm,v2,06/33] kasan: introduce kasan_print_aux_stacks [mm,v2,07/33] kasan: introduce kasan_get_alloc_track [mm,v2,08/33] kasan: introduce kasan_init_object_meta [mm,v2,09/33] kasan: clear metadata functions for tag-based modes [mm,v2,10/33] kasan: move kasan_get_*_meta to generic.c [mm,v2,11/33] kasan: introduce kasan_requires_meta [mm,v2,12/33] kasan: introduce kasan_init_cache_meta [mm,v2,13/33] kasan: drop CONFIG_KASAN_GENERIC check from kasan_init_cache_meta [mm,v2,14/33] kasan: only define kasan_metadata_size for Generic mode [mm,v2,15/33] kasan: only define kasan_never_merge for Generic mode [mm,v2,16/33] kasan: only define metadata offsets for Generic mode [mm,v2,17/33] kasan: only define metadata structs for Generic mode [mm,v2,18/33] kasan: only define kasan_cache_create for Generic mode [mm,v2,19/33] kasan: pass tagged pointers to kasan_save_alloc/free_info [mm,v2,20/33] kasan: move kasan_get_alloc/free_track definitions [mm,v2,21/33] kasan: cosmetic changes in report.c [mm,v2,22/33] kasan: use virt_addr_valid in kasan_addr_to_page/slab [mm,v2,23/33] kasan: use kasan_addr_to_slab in print_address_description [mm,v2,24/33] kasan: make kasan_addr_to_page static [mm,v2,25/33] kasan: simplify print_report [mm,v2,26/33] kasan: introduce complete_report_info [mm,v2,27/33] kasan: fill in cache and object in complete_report_info [mm,v2,28/33] kasan: rework function arguments in report.c [mm,v2,29/33] kasan: introduce kasan_complete_mode_report_info [mm,v2,30/33] kasan: implement stack ring for tag-based modes [mm,v2,31/33] kasan: support kasan.stacktrace for SW_TAGS [mm,v2,32/33] kasan: dynamically allocate stack ring entries [mm,v2,33/33] kasan: better identify bug types for tag-based modes

Commit Message

andrey.konovalov@linux.dev July 19, 2022, 12:10 a.m. UTC

From: Andrey Konovalov <andreyknvl@google.com>

To simplify reading the implementation of print_report(), remove the
tagged_addr variable and rename untagged_addr to addr.

Signed-off-by: Andrey Konovalov <andreyknvl@google.com>
---
 mm/kasan/report.c | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff mbox series

Patch

diff --git a/mm/kasan/report.c b/mm/kasan/report.c
index ac526c10ebff..dc38ada86f85 100644
--- a/mm/kasan/report.c
+++ b/mm/kasan/report.c
@@ -397,17 +397,16 @@  static void print_memory_metadata(const void *addr)
 
 static void print_report(struct kasan_report_info *info)
 {
-	void *tagged_addr = info->access_addr;
-	void *untagged_addr = kasan_reset_tag(tagged_addr);
-	u8 tag = get_tag(tagged_addr);
+	void *addr = kasan_reset_tag(info->access_addr);
+	u8 tag = get_tag(info->access_addr);
 
 	print_error_description(info);
-	if (addr_has_metadata(untagged_addr))
+	if (addr_has_metadata(addr))
 		kasan_print_tags(tag, info->first_bad_addr);
 	pr_err("\n");
 
-	if (addr_has_metadata(untagged_addr)) {
-		print_address_description(untagged_addr, tag);
+	if (addr_has_metadata(addr)) {
+		print_address_description(addr, tag);
 		print_memory_metadata(info->first_bad_addr);
 	} else {
 		dump_stack_lvl(KERN_ERR);