You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by lq...@apache.org on 2017/04/06 09:45:11 UTC
svn commit: r1790369 -
/qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/model/port/AbstractPort.java
Author: lquack
Date: Thu Apr 6 09:45:11 2017
New Revision: 1790369
URL: http://svn.apache.org/viewvc?rev=1790369&view=rev
Log:
QPID-7735: [Java Broker] Allow multiple ports to be configured using dynamic port 0
Modified:
qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/model/port/AbstractPort.java
Modified: qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/model/port/AbstractPort.java
URL: http://svn.apache.org/viewvc/qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/model/port/AbstractPort.java?rev=1790369&r1=1790368&r2=1790369&view=diff
==============================================================================
--- qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/model/port/AbstractPort.java (original)
+++ qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/model/port/AbstractPort.java Thu Apr 6 09:45:11 2017
@@ -183,11 +183,11 @@ public abstract class AbstractPort<X ext
if(changedAttributes.contains(PORT))
{
int newPort = updated.getPort();
- if (getPort() != newPort)
+ if (getPort() != newPort && newPort != 0)
{
for (Port p : _container.getChildren(Port.class))
{
- if (p.getPort() == newPort)
+ if (p.getBoundPort() == newPort || p.getPort() == newPort)
{
throw new IllegalConfigurationException("Port number "
+ newPort
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org