Merge pull request #4398 from mitchellh/portinclusive

step_http_server: make port range inclusive
2017-01-19 13:38:27 -08:00 · 2017-01-19 13:38:27 -08:00 · 56115ee270
parent efa48eeba8 36b436b2b7
commit 56115ee270
1 changed files with 2 additions and 1 deletions
--- a/common/step_http_server.go
+++ b/common/step_http_server.go
@ -49,7 +49,8 @@ func (s *StepHTTPServer) Run(state multistep.StateBag) multistep.StepAction {

 		if portRange > 0 {
 			// Intn will panic if portRange == 0, so we do a check.
-			offset = uint(rand.Intn(portRange))
+			// Intn is from [0, n), so add 1 to make from [0, n]
+			offset = uint(rand.Intn(portRange + 1))
 		}

 		httpPort = offset + s.HTTPPortMin