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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sun, 23 Mar 2014 21:14:56 -0700 From: Guenter Roeck <linux@...ck-us.net> To: linux-pm@...r.kernel.org Cc: linux-kernel@...r.kernel.org, Zhang Rui <rui.zhang@...el.com>, Eduardo Valentin <eduardo.valentin@...com>, Guenter Roeck <linux@...ck-us.net> Subject: [PATCH] thermal: Fix KELVIN_TO_CELSIUS macro It is always a good idea to use paranthesis around macro parameters to avoid undesired side effects. In this specific case, KELVIN_TO_CELSIUS() is used in drivers/platform/x86/asus-wmi.c with parameter "value & 0xFFFF", which due to operator evaluation order causes more or less random results. Signed-off-by: Guenter Roeck <linux@...ck-us.net> --- include/linux/thermal.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/linux/thermal.h b/include/linux/thermal.h index f7e11c7..02488c9 100644 --- a/include/linux/thermal.h +++ b/include/linux/thermal.h @@ -41,8 +41,8 @@ #define THERMAL_NO_LIMIT THERMAL_CSTATE_INVALID /* Unit conversion macros */ -#define KELVIN_TO_CELSIUS(t) (long)(((long)t-2732 >= 0) ? \ - ((long)t-2732+5)/10 : ((long)t-2732-5)/10) +#define KELVIN_TO_CELSIUS(t) (long)(((long)(t)-2732 >= 0) ? \ + ((long)(t)-2732+5)/10 : ((long)(t)-2732-5)/10) #define CELSIUS_TO_KELVIN(t) ((t)*10+2732) /* Adding event notification support elements */ -- 1.7.9.7 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists