brooklyn-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From mattunderscorechampion <>
Subject [GitHub] incubator-brooklyn pull request: Release ports acquired when check...
Date Wed, 28 Jan 2015 11:24:04 GMT
GitHub user mattunderscorechampion reopened a pull request:

    Release ports acquired when checking available

    Fixes issue [BROOKLYN-130]( Ensures
that the setReuseAddress is called before the port is bound. Removes the attempt to connect
to the port before binding it as unnecessary. Corrected off by one error in loop so that port
is not incremented beyond MAX_PORT_NUMBER.

You can merge this pull request into a Git repository by running:

    $ git pull correctPortRelease

Alternatively you can review and apply these changes as the patch at:

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #480
commit 0b5a77d4f3f8b4e5c1975d67c9cfddbab8a629eb
Author: Matt Champion <>
Date:   2015-01-27T17:20:51Z

    Bind the socket after setting SO_REUSEADDR to true, behaviour is undefined the other way

commit 5004bfee4c1ad6f98f3a61b9f697022ef59de70f
Author: Matt Champion <>
Date:   2015-01-27T20:40:30Z

    Ensure that the datagram socket is created in an unbound state

commit 34439f4a2ac0c13781a8485c4d6d308fb744de47
Author: Matt Champion <>
Date:   2015-01-27T20:43:58Z

    Removed unnecessary double checking of free port

commit f673e19910eaf160b2ba1c93e0d6a0d23a265f27
Author: Matt Champion <>
Date:   2015-01-27T20:58:43Z

    Fixed off by one error. After MAX_PORT_NUMBER comparison port is incremented and checked.
If not in valid range isPortAvailable throws an exception which is less descriptive than the
one in nextAvailablePort.


If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at or file a JIRA ticket
with INFRA.

View raw message