config: fix a possible crash in the monitor config parser (#9460)

* Less crash-prone monitor config parser * clang-format
hyprwm · Feb 23, 2025 · e594646 · e594646
1 parent 2e81648
commit e594646
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/src/config/ConfigManager.cpp b/src/config/ConfigManager.cpp
@@ -1946,11 +1946,16 @@ std::optional<std::string> CConfigManager::handleMonitor(const std::string& comm
             error += "invalid resolution ";
             newrule.resolution = Vector2D();
         } else {
-            newrule.resolution.x = stoi(ARGS[1].substr(0, ARGS[1].find_first_of('x')));
-            newrule.resolution.y = stoi(ARGS[1].substr(ARGS[1].find_first_of('x') + 1, ARGS[1].find_first_of('@')));
+            try {
+                newrule.resolution.x = stoi(ARGS[1].substr(0, ARGS[1].find_first_of('x')));
+                newrule.resolution.y = stoi(ARGS[1].substr(ARGS[1].find_first_of('x') + 1, ARGS[1].find_first_of('@')));
 
-            if (ARGS[1].contains("@"))
-                newrule.refreshRate = stof(ARGS[1].substr(ARGS[1].find_first_of('@') + 1));
+                if (ARGS[1].contains("@"))
+                    newrule.refreshRate = stof(ARGS[1].substr(ARGS[1].find_first_of('@') + 1));
+            } catch (...) {
+                error += "invalid resolution ";
+                newrule.resolution = Vector2D();
+            }
         }
     }