changeset 7119:9bc237efa31a

fix unitialized variables
author Sebastien Jodogne <s.jodogne@gmail.com>
date Fri, 14 Aug 2026 12:53:54 +0200
parents 50503cd4e588
children 0a8b8890f26d
files OrthancFramework/Sources/DataSource/DataSourceMemoryBudget.cpp OrthancFramework/Sources/DataSource/DataSourceMemoryBudget.h
diffstat 2 files changed, 9 insertions(+), 5 deletions(-) [+]
line wrap: on
line diff
--- a/OrthancFramework/Sources/DataSource/DataSourceMemoryBudget.cpp	Fri Aug 14 12:03:59 2026 +0200
+++ b/OrthancFramework/Sources/DataSource/DataSourceMemoryBudget.cpp	Fri Aug 14 12:53:54 2026 +0200
@@ -69,10 +69,13 @@
     {
       maxMemoryUsageSinceStart_ = std::max(currentMemorySize, maxMemoryUsageSinceStart_);
 
-      metrics_->SetFloatValue(capacityMaxSizeMegabytesName_, BytesToFloatMegabytes(maximumMemorySize)); // will be updated when we set the capacity
-      metrics_->SetFloatValue(capacityCurrentSizeMegabytesName_, BytesToFloatMegabytes(currentMemorySize));
-      metrics_->SetIntegerValue(capacityCountName_, currentReservationCount);
-      metrics_->SetFloatValue(capacityMaxUsageSinceStartMegabytesName_, BytesToFloatMegabytes(maxMemoryUsageSinceStart_));
+      if (metrics_)
+      {
+        metrics_->SetFloatValue(capacityMaxSizeMegabytesName_, BytesToFloatMegabytes(maximumMemorySize)); // will be updated when we set the capacity
+        metrics_->SetFloatValue(capacityCurrentSizeMegabytesName_, BytesToFloatMegabytes(currentMemorySize));
+        metrics_->SetIntegerValue(capacityCountName_, currentReservationCount);
+        metrics_->SetFloatValue(capacityMaxUsageSinceStartMegabytesName_, BytesToFloatMegabytes(maxMemoryUsageSinceStart_));
+      }
     }
 
 
--- a/OrthancFramework/Sources/DataSource/DataSourceMemoryBudget.h	Fri Aug 14 12:03:59 2026 +0200
+++ b/OrthancFramework/Sources/DataSource/DataSourceMemoryBudget.h	Fri Aug 14 12:53:54 2026 +0200
@@ -63,7 +63,8 @@
         uint64_t                            maxMemoryUsageSinceStart_;
 
       public:
-        MetricsConfiguration()
+        MetricsConfiguration() :
+          maxMemoryUsageSinceStart_(0)
         {
         }