Add length check

bbimber · bbimber · commit 301768be0890 · 2025-06-13T09:04:33.000-07:00
diff --git a/cluster/src/org/labkey/cluster/pipeline/SlurmExecutionEngine.java b/cluster/src/org/labkey/cluster/pipeline/SlurmExecutionEngine.java
@@ -499,6 +499,14 @@ private String extractField(String line, List<String> fieldWidths, int idx)
 
         try
         {
+            if (end > line.length())
+            {
+                _log.debug("Encountered slurm line shorter than expected. Slurm field at idx: " + idx + ". Line: [" + line + "]");
+                _log.debug("fieldWidths: " + StringUtils.join(fieldWidths, "|"));
+
+                return null;
+            }
+
             return StringUtils.trimToNull(line.substring(start, end));
         }
         catch (Exception e)

Original file line number	Diff line number	Diff line change
`@@ -499,6 +499,14 @@ private String extractField(String line, List<String> fieldWidths, int idx)`
`499`	`499`
`500`	`500`	`try`
`501`	`501`	`{`
	`502`	`+ if (end > line.length())`
	`503`	`+ {`
	`504`	`+ _log.debug("Encountered slurm line shorter than expected. Slurm field at idx: " + idx + ". Line: [" + line + "]");`
	`505`	`+ _log.debug("fieldWidths: " + StringUtils.join(fieldWidths, "\|"));`
	`506`	`+`
	`507`	`+ return null;`
	`508`	`+ }`
	`509`	`+`
`502`	`510`	`return StringUtils.trimToNull(line.substring(start, end));`
`503`	`511`	`}`
`504`	`512`	`catch (Exception e)`