linux-kernel - Re: UBSAN: Undefined behaviour in arch/powerpc/kernel/cputable.c

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>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <alpine.LRH.2.20.1609111211580.26664@math.ut.ee>
Date:   Sun, 11 Sep 2016 12:12:22 +0300 (EEST)
From:   Meelis Roos <mroos@...ux.ee>
To:     Michael Ellerman <mpe@...erman.id.au>
cc:     Linux Kernel list <linux-kernel@...r.kernel.org>,
        "Aneesh Kumar K.V" <aneesh.kumar@...ux.vnet.ibm.com>,
        linuxppc-dev@...ts.ozlabs.org
Subject: Re: UBSAN: Undefined behaviour in arch/powerpc/kernel/cputable.c

> Does this fix it?

Yes, thank you!

> diff --git a/arch/powerpc/include/asm/cpu_has_feature.h b/arch/powerpc/include/asm/cpu_has_feature.h
> index 2ef55f8968a2..b312b152461b 100644
> --- a/arch/powerpc/include/asm/cpu_has_feature.h
> +++ b/arch/powerpc/include/asm/cpu_has_feature.h
> @@ -15,7 +15,7 @@ static inline bool early_cpu_has_feature(unsigned long feature)
>  #ifdef CONFIG_JUMP_LABEL_FEATURE_CHECKS
>  #include <linux/jump_label.h>
>  
> -#define NUM_CPU_FTR_KEYS	64
> +#define NUM_CPU_FTR_KEYS	BITS_PER_LONG
>  
>  extern struct static_key_true cpu_feature_keys[NUM_CPU_FTR_KEYS];
>  
> 

-- 
Meelis Roos (mroos@...ux.ee)