Qpid
  1. Qpid
  2. QPID-4671

[Java Broker] NPE thrown during "exchange.bound" command against a fanout exchange when specifying the name of a queue that does not exist

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.18, 0.20
    • Fix Version/s: 0.22
    • Component/s: Java Broker
    • Labels:
      None

      Description

      [Java Broker] NPE is thrown on processing of "exchange.bound" command having exchange argument set to a fanout exchange and queue argument set non-existing queue:

      (ServerSessionDelegate.java:119) - Exception processing command
      java.lang.NullPointerException
      at java.util.concurrent.ConcurrentHashMap.containsKey(ConcurrentHashMap.java:782)
      at org.apache.qpid.server.exchange.FanoutExchange.isBound(FanoutExchange.java:111)
      at org.apache.qpid.server.exchange.AbstractExchange.isBound(AbstractExchange.java:182)
      at org.apache.qpid.server.transport.ServerSessionDelegate.exchangeBound(ServerSessionDelegate.java:1159)
      at org.apache.qpid.server.transport.ServerSessionDelegate.exchangeBound(ServerSessionDelegate.java:74)
      at org.apache.qpid.transport.ExchangeBound.dispatch(ExchangeBound.java:99)
      at org.apache.qpid.transport.SessionDelegate.command(SessionDelegate.java:55)
      at org.apache.qpid.server.transport.ServerSessionDelegate.command(ServerSessionDelegate.java:98)
      at org.apache.qpid.server.transport.ServerSessionDelegate.command(ServerSessionDelegate.java:74)
      at org.apache.qpid.transport.Method.delegate(Method.java:159)
      at org.apache.qpid.transport.Session.received(Session.java:584)
      at org.apache.qpid.transport.Connection.dispatch(Connection.java:421)
      at org.apache.qpid.transport.ConnectionDelegate.handle(ConnectionDelegate.java:64)
      at org.apache.qpid.transport.ConnectionDelegate.handle(ConnectionDelegate.java:40)
      at org.apache.qpid.transport.MethodDelegate.exchangeBound(MethodDelegate.java:203)
      at org.apache.qpid.transport.ExchangeBound.dispatch(ExchangeBound.java:99)
      at org.apache.qpid.transport.ConnectionDelegate.command(ConnectionDelegate.java:54)
      at org.apache.qpid.transport.ConnectionDelegate.command(ConnectionDelegate.java:40)
      at org.apache.qpid.transport.Method.delegate(Method.java:159)
      at org.apache.qpid.transport.Connection.received(Connection.java:376)
      at org.apache.qpid.server.transport.ServerConnection.received(ServerConnection.java:212)
      at org.apache.qpid.server.transport.ServerConnection.received(ServerConnection.java:56)
      at org.apache.qpid.transport.network.Assembler.emit(Assembler.java:97)
      at org.apache.qpid.transport.network.Assembler.assemble(Assembler.java:198)
      at org.apache.qpid.transport.network.Assembler.frame(Assembler.java:131)
      at org.apache.qpid.transport.network.Frame.delegate(Frame.java:128)
      at org.apache.qpid.transport.network.Assembler.received(Assembler.java:102)
      at org.apache.qpid.transport.network.Assembler.received(Assembler.java:44)
      at org.apache.qpid.transport.network.InputHandler.next(InputHandler.java:189)
      at org.apache.qpid.transport.network.InputHandler.received(InputHandler.java:105)
      at org.apache.qpid.server.protocol.ProtocolEngine_0_10.received(ProtocolEngine_0_10.java:109)
      at org.apache.qpid.server.protocol.ProtocolEngine_0_10.received(ProtocolEngine_0_10.java:42)
      at org.apache.qpid.server.protocol.MultiVersionProtocolEngine.received(MultiVersionProtocolEngine.java:118)
      at org.apache.qpid.server.protocol.MultiVersionProtocolEngine.received(MultiVersionProtocolEngine.java:37)
      at org.apache.qpid.transport.network.io.IoReceiver.run(IoReceiver.java:152)
      at java.lang.Thread.run(Thread.java:662)

        Activity

        Hide
        Robbie Gemmell added a comment -

        Now merged to the 0.22 release branch:
        http://svn.apache.org/r1462778

        Show
        Robbie Gemmell added a comment - Now merged to the 0.22 release branch: http://svn.apache.org/r1462778
        Hide
        Justin Ross added a comment -

        Reviewed by Robbie. Approved for 0.22.

        Show
        Justin Ross added a comment - Reviewed by Robbie. Approved for 0.22.
        Hide
        Robbie Gemmell added a comment -

        Looks good to me. We should request for 0.22.

        Show
        Robbie Gemmell added a comment - Looks good to me. We should request for 0.22.
        Hide
        Alex Rudyy added a comment -

        Robbie,
        Could you please review the changes in http://svn.apache.org/r1462162 ?

        Show
        Alex Rudyy added a comment - Robbie, Could you please review the changes in http://svn.apache.org/r1462162 ?

          People

          • Assignee:
            Robbie Gemmell
            Reporter:
            Alex Rudyy
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development