summaryrefslogtreecommitdiff
path: root/bench
diff options
context:
space:
mode:
authorBenoit Daloze <eregontp@gmail.com>2020-03-29 14:25:18 +0200
committerBenoit Daloze <eregontp@gmail.com>2020-03-29 14:25:29 +0200
commitbf2e8afcfadd38300c3f0f55ecf3042e9996f34e (patch)
tree4d232c8c3acfa0bac6c8f1b5178f036499e945f9 /bench
parentc0ad8036f1a6a2dd7c8eb6bc42c2fa2b1e04b0da (diff)
downloadffi-bf2e8afcfadd38300c3f0f55ecf3042e9996f34e.tar.gz
Relax check in bench_getlogin.rb
Diffstat (limited to 'bench')
-rw-r--r--bench/bench_getlogin.rb5
1 files changed, 4 insertions, 1 deletions
diff --git a/bench/bench_getlogin.rb b/bench/bench_getlogin.rb
index 12233af..8cb5176 100644
--- a/bench/bench_getlogin.rb
+++ b/bench/bench_getlogin.rb
@@ -9,7 +9,10 @@ module BenchGetlogin
ffi_lib FFI::Library::LIBC
attach_function :getlogin, [], :string
end
- if Posix.getlogin != Etc.getlogin
+
+ # getlogin(2) might return NULL, in which case Etc.getlogin uses ENV["USER"].
+ # This should not matter for this benchmark, getlogin() is still called.
+ if (Posix.getlogin || ENV["USER"]) != Etc.getlogin
raise ArgumentError, "FFI getlogin returned incorrect value: " \
"#{Posix.getlogin.inspect} (FFI) vs #{Etc.getlogin.inspect} (Etc)"
end