aboutsummaryrefslogtreecommitdiffstats
path: root/erts/emulator/sys
diff options
context:
space:
mode:
authorSverker Eriksson <[email protected]>2018-07-06 20:12:28 +0200
committerSverker Eriksson <[email protected]>2018-07-10 14:10:25 +0200
commit7603a4fb82f72a72045eed26dd55de3067cfe344 (patch)
tree5dbbd3aa5e0e97a2484ad125508e80d33561c3da /erts/emulator/sys
parent4100c0cb4d391693fdb08d8565c937bb399fda3a (diff)
downloadotp-7603a4fb82f72a72045eed26dd55de3067cfe344.tar.gz
otp-7603a4fb82f72a72045eed26dd55de3067cfe344.tar.bz2
otp-7603a4fb82f72a72045eed26dd55de3067cfe344.zip
erts: Fix buggy calls to erts_sys_explicit_8bit_getenv
Two of them only affect valgrind builds and the one for ERL_CRASH_DUMP_NICE seems benign. Return value changed in c2d70945dce9cb09d5d7120d6e9ddf7faac8d230 old -> new -1 -> 0 not found 0 -> 1 found ok 1 -> -1 found but too big
Diffstat (limited to 'erts/emulator/sys')
-rw-r--r--erts/emulator/sys/unix/sys.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/erts/emulator/sys/unix/sys.c b/erts/emulator/sys/unix/sys.c
index 189ca083d7..36579ffdb4 100644
--- a/erts/emulator/sys/unix/sys.c
+++ b/erts/emulator/sys/unix/sys.c
@@ -452,9 +452,9 @@ prepare_crash_dump(int secs)
envsz = sizeof(env);
i = erts_sys_explicit_8bit_getenv("ERL_CRASH_DUMP_NICE", env, &envsz);
- if (i >= 0) {
+ if (i != 0) {
int nice_val;
- nice_val = i != 1 ? 0 : atoi(env);
+ nice_val = (i != 1) ? 0 : atoi(env);
if (nice_val > 39) {
nice_val = 39;
}