diff mbox series

[mm,v2,21/33] kasan: cosmetic changes in report.c

Message ID	7b5f4b94b922c1753190886d0b6984bc1c16828f.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 21/33] kasan: cosmetic changes in report.c Date: Tue, 19 Jul 2022 02:10:01 +0200 Message-Id: <7b5f4b94b922c1753190886d0b6984bc1c16828f.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>

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

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;
 }