Test for null, avoid NPE

fix #2280
OpenTSDB · Jul 10, 2023 · 9de0f9f · 9de0f9f
1 parent fd78f0e
commit 9de0f9f
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/src/tsd/GraphHandler.java b/src/tsd/GraphHandler.java
@@ -204,10 +204,12 @@ private void doGraph(final TSDB tsdb, final HttpQuery query)
       }
 
       options = query.getQueryStringParams("o");
-      for (int i = 0; i < options.size(); i++) {
-        if (!allow_list.contains(options.get(i))) {
-          throw new BadRequestException("Query option at index " + i 
-              + " was not in the allow list.");
+      if (!(options == null)) {
+        for (int i = 0; i < options.size(); i++) {
+          if (!allow_list.contains(options.get(i))) {
+            throw new BadRequestException("Query option at index " + i 
+                + " was not in the allow list.");
+          }
         }
       }
     }