From 2b6c00db3406c753291ff00ccd16d64ad5f92a78 Mon Sep 17 00:00:00 2001 From: Daniel Beck Date: Fri, 3 Oct 2014 16:07:24 +0200 Subject: [PATCH] [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. --- debian/debian/jenkins.init | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/debian/debian/jenkins.init b/debian/debian/jenkins.init index 669b4f39d2..2ec7789531 100644 --- a/debian/debian/jenkins.init +++ b/debian/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