diff mbox series

[mm,v3,21/34] kasan: cosmetic changes in report.c

Message ID	b728eae71f3ea505a885449724de21cf3f476a7b.1662411799.git.andreyknvl@google.com (mailing list archive)
State	New
Headers	show Return-Path: <owner-linux-mm@kvack.org> From: andrey.konovalov@linux.dev To: Andrew Morton <akpm@linux-foundation.org> Cc: Andrey Konovalov <andreyknvl@gmail.com>, Marco Elver <elver@google.com>, Alexander Potapenko <glider@google.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>, linux-mm@kvack.org, linux-kernel@vger.kernel.org, Andrey Konovalov <andreyknvl@google.com> Subject: [PATCH mm v3 21/34] kasan: cosmetic changes in report.c Date: Mon, 5 Sep 2022 23:05:36 +0200 Message-Id: <b728eae71f3ea505a885449724de21cf3f476a7b.1662411799.git.andreyknvl@google.com> In-Reply-To: <cover.1662411799.git.andreyknvl@google.com> References: <cover.1662411799.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,v3,00/34] kasan: switch tag-based modes to stack ring from per-object metadata [mm,v3,01/34] kasan: check KASAN_NO_FREE_META in __kasan_metadata_size [mm,v3,02/34] kasan: rename kasan_set__info to kasan_save__info [mm,v3,03/34] kasan: move is_kmalloc check out of save_alloc_info [mm,v3,04/34] kasan: split save_alloc_info implementations [mm,v3,05/34] kasan: drop CONFIG_KASAN_TAGS_IDENTIFY [mm,v3,06/34] kasan: introduce kasan_print_aux_stacks [mm,v3,07/34] kasan: introduce kasan_get_alloc_track [mm,v3,08/34] kasan: introduce kasan_init_object_meta [mm,v3,09/34] kasan: clear metadata functions for tag-based modes [mm,v3,10/34] kasan: move kasan_get_*_meta to generic.c [mm,v3,11/34] kasan: introduce kasan_requires_meta [mm,v3,12/34] kasan: introduce kasan_init_cache_meta [mm,v3,13/34] kasan: drop CONFIG_KASAN_GENERIC check from kasan_init_cache_meta [mm,v3,14/34] kasan: only define kasan_metadata_size for Generic mode [mm,v3,15/34] kasan: only define kasan_never_merge for Generic mode [mm,v3,16/34] kasan: only define metadata offsets for Generic mode [mm,v3,17/34] kasan: only define metadata structs for Generic mode [mm,v3,18/34] kasan: only define kasan_cache_create for Generic mode [mm,v3,19/34] kasan: pass tagged pointers to kasan_save_alloc/free_info [mm,v3,20/34] kasan: move kasan_get_alloc/free_track definitions [mm,v3,21/34] kasan: cosmetic changes in report.c [mm,v3,22/34] kasan: use virt_addr_valid in kasan_addr_to_page/slab [mm,v3,23/34] kasan: use kasan_addr_to_slab in print_address_description [mm,v3,24/34] kasan: make kasan_addr_to_page static [mm,v3,25/34] kasan: simplify print_report [mm,v3,26/34] kasan: introduce complete_report_info [mm,v3,27/34] kasan: fill in cache and object in complete_report_info [mm,v3,28/34] kasan: rework function arguments in report.c [mm,v3,29/34] kasan: introduce kasan_complete_mode_report_info [mm,v3,30/34] kasan: implement stack ring for tag-based modes [mm,v3,31/34] kasan: support kasan.stacktrace for SW_TAGS [mm,v3,32/34] kasan: dynamically allocate stack ring entries [mm,v3,33/34] kasan: better identify bug types for tag-based modes [mm,v3,34/34] kasan: add another use-after-free test

Commit Message

andrey.konovalov@linux.dev Sept. 5, 2022, 9:05 p.m. UTC

From: Andrey Konovalov <andreyknvl@google.com>

Do a few non-functional style fixes for the code in report.c.

Reviewed-by: Marco Elver <elver@google.com>
Signed-off-by: Andrey Konovalov <andreyknvl@google.com>
---
 mm/kasan/report.c | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

diff mbox series

Patch

diff --git a/mm/kasan/report.c b/mm/kasan/report.c
index 5d225d7d9c4c..83f420a28c0b 100644
--- a/mm/kasan/report.c
+++ b/mm/kasan/report.c
@@ -200,25 +200,22 @@  static void print_error_description(struct kasan_report_info *info)
 static void print_track(struct kasan_track *track, const char *prefix)
 {
 	pr_err("%s by task %u:\n", prefix, track->pid);
-	if (track->stack) {
+	if (track->stack)
 		stack_depot_print(track->stack);
-	} else {
+	else
 		pr_err("(stack is not available)\n");
-	}
 }
 
 struct page *kasan_addr_to_page(const void *addr)
 {
-	if ((addr >= (void *)PAGE_OFFSET) &&
-			(addr < high_memory))
+	if ((addr >= (void *)PAGE_OFFSET) && (addr < high_memory))
 		return virt_to_head_page(addr);
 	return NULL;
 }
 
 struct slab *kasan_addr_to_slab(const void *addr)
 {
-	if ((addr >= (void *)PAGE_OFFSET) &&
-			(addr < high_memory))
+	if ((addr >= (void *)PAGE_OFFSET) && (addr < high_memory))
 		return virt_to_slab(addr);
 	return NULL;
 }