Fix the MetricLockGuard::owns and its use to match the intention.

2 files changed, 2 insertions, 2 deletions

diff --git a/metrics/src/vespa/metrics/metricmanager.cpp b/metrics/src/vespa/metrics/metricmanager.cpp
index 6447e12a0d2..8ca74384af0 100644
--- a/metrics/src/vespa/metrics/metricmanager.cpp
+++ b/metrics/src/vespa/metrics/metricmanager.cpp
@@ -32,7 +32,7 @@ MetricManager::Timer::getTime() const {
 
 void
 MetricManager::assertMetricLockLocked(const MetricLockGuard& g) const {
-    if (g.owns(_waiter)) {
+    if ( ! g.owns(_waiter)) {
         throw vespalib::IllegalArgumentException("Given lock does not lock the metric lock.", VESPA_STRLOC);
     }
 }
diff --git a/metrics/src/vespa/metrics/updatehook.cpp b/metrics/src/vespa/metrics/updatehook.cpp
index 99051e8df64..b627d55b09e 100644
--- a/metrics/src/vespa/metrics/updatehook.cpp
+++ b/metrics/src/vespa/metrics/updatehook.cpp
@@ -10,7 +10,7 @@ MetricLockGuard::MetricLockGuard(std::mutex & mutex)
 
 bool
 MetricLockGuard::owns(const std::mutex & mutex) const {
-    return (_guard.mutex() == &mutex) && !_guard.owns_lock();
+    return (_guard.mutex() == &mutex) && _guard.owns_lock();
 }
 
 MetricLockGuard::~MetricLockGuard() = default;