git ls-remote may return null when finding PR num (#7)

zanata · Nov 15, 2017 · 5c612d8 · 5c612d8
1 parent 9c54e1a
commit 5c612d8
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/src/org/zanata/jenkins/ScmGit.groovy b/src/org/zanata/jenkins/ScmGit.groovy
@@ -65,6 +65,7 @@ class ScmGit implements Serializable {
 
     // Find PR Num
     if (!pullGitLsRemoteLines) {
+      // git ls-remote may return null here
       pullGitLsRemoteLines = steps.sh([
         returnStdout: true,
         script: "git ls-remote $repoUrl refs/pull/*/head",
@@ -95,6 +96,11 @@ class ScmGit implements Serializable {
     for (int i = 0; i < gitLsRemoteLines.length; i++) {
       // split each line by whitespace
       String[] tokens = gitLsRemoteLines[i].split()
+      if (tokens.length < 2) {
+        // It is possible gitLsRemoteLines contains [''],
+        // an array that contains one empty string
+        continue
+      }
       if (tokens[0] == commitId) {
         // format to search is refs/heads/<branch>
         return tokens[1].split('/')[2]