lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [day] [month] [year] [list]
Message-ID: <173058261037.3137.8690137124888546964.tip-bot2@tip-bot2>
Date: Sat, 02 Nov 2024 21:23:30 -0000
From: "tip-bot2 for Thomas Gleixner" <tip-bot2@...utronix.de>
To: linux-tip-commits@...r.kernel.org
Cc: kernel test robot <lkp@...el.com>, Thomas Gleixner <tglx@...utronix.de>,
 x86@...nel.org, linux-kernel@...r.kernel.org
Subject: [tip: timers/vdso] x86/vdso: Add missing brackets in switch case

The following commit has been merged into the timers/vdso branch of tip:

Commit-ID:     7fa3c36ea2707c495cf31ccab733ac8bf3f9d0c2
Gitweb:        https://git.kernel.org/tip/7fa3c36ea2707c495cf31ccab733ac8bf3f9d0c2
Author:        Thomas Gleixner <tglx@...utronix.de>
AuthorDate:    Sat, 02 Nov 2024 22:11:24 +01:00
Committer:     Thomas Gleixner <tglx@...utronix.de>
CommitterDate: Sat, 02 Nov 2024 22:16:14 +01:00

x86/vdso: Add missing brackets in switch case

0-day reported:

  arch/x86/entry/vdso/vma.c:199:3: warning: label followed by a declaration
  				 	    is a C23 extension [-Wc23-extensions]

Add the missing brackets.

Fixes: e93d2521b27f ("x86/vdso: Split virtual clock pages into dedicated mapping")
Reported-by: kernel test robot <lkp@...el.com>
Signed-off-by: Thomas Gleixner <tglx@...utronix.de>
Closes: https://lore.kernel.org/oe-kbuild-all/202411022359.fBPFTg2T-lkp@intel.com/
---
 arch/x86/entry/vdso/vma.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/arch/x86/entry/vdso/vma.c b/arch/x86/entry/vdso/vma.c
index 7e5921a..bfc7cab 100644
--- a/arch/x86/entry/vdso/vma.c
+++ b/arch/x86/entry/vdso/vma.c
@@ -196,21 +196,26 @@ static vm_fault_t vvar_vclock_fault(const struct vm_special_mapping *sm,
 	switch (vmf->pgoff) {
 #ifdef CONFIG_PARAVIRT_CLOCK
 	case VDSO_PAGE_PVCLOCK_OFFSET:
+	{
 		struct pvclock_vsyscall_time_info *pvti =
 			pvclock_get_pvti_cpu0_va();
+
 		if (pvti && vclock_was_used(VDSO_CLOCKMODE_PVCLOCK))
 			return vmf_insert_pfn_prot(vma, vmf->address,
 					__pa(pvti) >> PAGE_SHIFT,
 					pgprot_decrypted(vma->vm_page_prot));
 		break;
+	}
 #endif /* CONFIG_PARAVIRT_CLOCK */
 #ifdef CONFIG_HYPERV_TIMER
 	case VDSO_PAGE_HVCLOCK_OFFSET:
+	{
 		unsigned long pfn = hv_get_tsc_pfn();
 
 		if (pfn && vclock_was_used(VDSO_CLOCKMODE_HVCLOCK))
 			return vmf_insert_pfn(vma, vmf->address, pfn);
 		break;
+	}
 #endif /* CONFIG_HYPERV_TIMER */
 	}
 

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ