Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[FIXED JENKINS-9281] Match entire port number instead of just prefix …
…using POSIX character class.
  • Loading branch information
lhanson authored and kohsuke committed Apr 13, 2011
1 parent 7cb015e commit 5061830
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
3 changes: 3 additions & 0 deletions changelog.html
Expand Up @@ -58,6 +58,9 @@
<li class=bug>
Some french strings are incorrect after renaming to Jenkins
(<a href="http://issues.jenkins-ci.org/browse/JENKINS-9334">issue 9334</a>)
<li class=bug>
Debian init script gives false positives for port already in use
(<a href="http://issues.jenkins-ci.org/browse/JENKINS-9281">issue 9281</a>)
<li class=rfe>
Added two new CLI commands "wait-node-online" and "wait-node-offline" to block until a slave becomes online/offline.
</ul>
Expand Down
2 changes: 1 addition & 1 deletion debian/debian/jenkins.init
Expand Up @@ -68,7 +68,7 @@ check_tcp_port() {
port=$default
fi

count=`netstat --listen --numeric-ports | grep \:$port | grep -c . `
count=`netstat --listen --numeric-ports | grep \:$port[[:space:]] | grep -c . `

if [ $count -ne 0 ]; then
echo "The selected $service port ($port) seems to be in use by another program "
Expand Down

0 comments on commit 5061830

Please sign in to comment.