Skip to content

[JENKINS-34309] Only parse reasonable branch line output#213

Merged
MarkEWaite merged 1 commit intojenkinsci:masterfrom
MarkEWaite:master-PRxxx-parseBranches-index-out-of-bounds
Jul 12, 2016
Merged

[JENKINS-34309] Only parse reasonable branch line output#213
MarkEWaite merged 1 commit intojenkinsci:masterfrom
MarkEWaite:master-PRxxx-parseBranches-index-out-of-bounds

Conversation

@MarkEWaite
Copy link
Contributor

Avoid string index out of bounds exception on unexpected output from
"git branch -v --no-abbrev" (as reported in JENKINS-34309).

The JENKINS-34309 issue may be related to gitlab bug 14453
https://gitlab.com/gitlab-org/gitlab-ce/issues/14453

Added a test to check that empty repo reports no branches. Empty
repo does not show the problem which was detected in JENKINS-34309.
My attempts to duplicate JENKINS-34309 in an automated test have all
failed. Those attempts are included in this commit (though unsuccessful
at duplicating the problem).

Avoid string index out of bounds exception on unexpected output from
"git branch -v --no-abbrev" (as reported in JENKINS-34309).

The JENKINS-34309 issue may be related to gitlab bug 14453
https://gitlab.com/gitlab-org/gitlab-ce/issues/14453

Added a test to check that empty repo reports no branches.  Empty
repo does not show the problem which was detected in JENKINS-34309.
My attempts to duplicate JENKINS-34309 in an automated test have all
failed.  Those attempts are included in this commit (though unsuccessful
at duplicating the problem).
@MarkEWaite MarkEWaite merged commit 8c44d39 into jenkinsci:master Jul 12, 2016
@MarkEWaite MarkEWaite deleted the master-PRxxx-parseBranches-index-out-of-bounds branch July 12, 2016 23:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant