Merge pull request #698 from rmcgibbo/range-check

range check getPlatform()

Merge pull request #698 from rmcgibbo/range-check
range check getPlatform()
071b6fa0 · peastman · 2bf08a95 · c1cbd441 · 071b6fa0
Commit 071b6fa0 authored Nov 04, 2014 by peastman
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 1 deletion

olla/src/Platform.cpp olla/src/Platform.cpp +4 -1

No files found.
--- a/olla/src/Platform.cpp
+++ b/olla/src/Platform.cpp
@@ -134,7 +134,10 @@ int Platform::getNumPlatforms() {
 }
 Platform& Platform::getPlatform(int index) {
-    return *getPlatforms()[index];
+    if (index >= 0 && index < getNumPlatforms()) {
+        return *getPlatforms()[index];
+    }
+    throw OpenMMException("Invalid platform index");
 }
 Platform& Platform::getPlatformByName(const string& name) {