From 03711ea5e53c7bca20f00d93cf90d37a937da1df Mon Sep 17 00:00:00 2001
From: w0j0pl <75681054+w0j0pl@users.noreply.github.com>
Date: Thu, 29 Aug 2024 15:15:09 +0200
Subject: [PATCH] Fix logic bug in cache_nvidia_string_value_update() (#2022)

* Update nvidia.cc

Made a suggested change in logic from issue #1177

* Update nvidia.cc

Made a suggested change in logic from issue #1178

* Update nvidia.cc

Made a suggested change in logic from issue #1178
---
 src/nvidia.cc | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/nvidia.cc b/src/nvidia.cc
index a7993c97c..763720511 100644
--- a/src/nvidia.cc
+++ b/src/nvidia.cc
@@ -829,11 +829,12 @@ void cache_nvidia_string_value_update(nvidia_c_string *ac_string, char *token,
              ac_string[gid].memTransferRatemax < 0) {
     ac_string[gid].memTransferRatemax = *value;
 
-  } else if (strcmp(token, (char *)"perf") == 0 &&
-             ac_string[gid].memTransferRatemax < 0) {
-    if (search == SEARCH_MIN) {
+  } else if (strcmp(token, (char *)"perf") == 0) {
+    if (search == SEARCH_MIN &&
+        ac_string[gid].perfmin < 0) {
       ac_string[gid].perfmin = *value;
-    } else if (search == SEARCH_MAX) {
+    } else if (search == SEARCH_MAX &&
+               ac_string[gid].perfmax < 0) {
       ac_string[gid].perfmax = *value;
     }
   }