[FIXED JENKINS-24966] Return failure when port is in use

Both 0 and 1 are used to indicate some kind of success (the latter when Jenkins doesn't need to be started because it was already running), so we need to return 2. Originally-From: jenkins-ci.org/commit/core/2b6c00db3406c753291ff00ccd16d64ad5f92a78
jenkinsci · Oct 3, 2014 · e28110f · e28110f
1 parent 055ee81
commit e28110f
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/debian/jenkins.init b/debian/jenkins.init
@@ -103,7 +103,7 @@ do_start()
 
     # Verify that the jenkins port is not already in use, winstone does not exit
     # even for BindException
-    check_tcp_port "http" "$HTTP_PORT" "8080" || return 1
+    check_tcp_port "http" "$HTTP_PORT" "8080" || return 2
 
     # If the var MAXOPENFILES is enabled in /etc/default/jenkins then set the max open files to the 
     # proper value