summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRichard Yao <[email protected]>2022-11-29 12:53:33 -0500
committerTony Hutter <[email protected]>2022-12-01 12:39:44 -0800
commit0e3abd29945b6d6a4f89892445f4d85bcb1d1e92 (patch)
tree4cd5a4e19eff61b1bd57c71ae7735f36a74eb334
parent3d1e8080960886cf9d542e318a400a185c521806 (diff)
Lua: Fix bad bitshift in lua_strx2number()
The port of lua to OpenZFS modified lua to use int64_t for numbers instead of double. As part of this, a function for calculating exponentiation was replaced with a bit shift. Unfortunately, it did not handle negative values. Also, it only supported exponents numbers with 7 digits before before overflow. This supports exponents up to 15 digits before overflow. Clang's static analyzer reported this as "Result of operation is garbage or undefined" because the exponent was negative. Reviewed-by: Damian Szuberski <[email protected]> Reviewed-by: Brian Behlendorf <[email protected]> Signed-off-by: Richard Yao <[email protected]> Closes #14204
-rw-r--r--module/lua/lobject.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/module/lua/lobject.c b/module/lua/lobject.c
index 024d3199f..5d88d36c5 100644
--- a/module/lua/lobject.c
+++ b/module/lua/lobject.c
@@ -144,7 +144,7 @@ static lua_Number lua_strx2number (const char *s, char **endptr) {
*endptr = cast(char *, s); /* valid up to here */
ret:
if (neg) r = -r;
- return (r * (1 << e));
+ return ((e >= 0) ? (r * (1ULL << e)) : (r / (1ULL << -e)));
}
#endif