Skip to content

B
benchmark
Commit 45b79c3e by Yusuke Suzuki
Options

Check families is not nullptr before using it

When there's no benchmarks, families becomes nullptr. So before touching it, we need to ensure families is not nullptr.
parent 254a6f9a
Showing with 3 additions and 0 deletions
AUTHORS
...@@ -14,3 +14,4 @@ Dominic Hamon <dma@stripysock.com> ...@@ -14,3 +14,4 @@ Dominic Hamon <dma@stripysock.com>
Eugene Zhuk <eugene.zhuk@gmail.com> Eugene Zhuk <eugene.zhuk@gmail.com>
Google Inc. Google Inc.
Oleksandr Sochka <sasha.sochka@gmail.com> Oleksandr Sochka <sasha.sochka@gmail.com>
Yusuke Suzuki <utatane.tea@gmail.com>
CONTRIBUTORS
...@@ -28,3 +28,4 @@ Dominic Hamon <dma@stripysock.com> ...@@ -28,3 +28,4 @@ Dominic Hamon <dma@stripysock.com>
Eugene Zhuk <eugene.zhuk@gmail.com> Eugene Zhuk <eugene.zhuk@gmail.com>
Oleksandr Sochka <sasha.sochka@gmail.com> Oleksandr Sochka <sasha.sochka@gmail.com>
Pierre Phaneuf <pphaneuf@google.com> Pierre Phaneuf <pphaneuf@google.com>
Yusuke Suzuki <utatane.tea@gmail.com>
src/benchmark.cc
...@@ -757,6 +757,7 @@ void Benchmark::FindBenchmarks(const std::string& spec, ...@@ -757,6 +757,7 @@ void Benchmark::FindBenchmarks(const std::string& spec,
} }
mutex_lock l(&benchmark_mutex); mutex_lock l(&benchmark_mutex);
if (families == nullptr) return; // There's no families.
for (Benchmark* family : *families) { for (Benchmark* family : *families) {
if (family == nullptr) continue; // Family was deleted if (family == nullptr) continue; // Family was deleted
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment