[I] [Bug] ConnectionPool::remove Crash [pulsar-client-cpp]

["arhoads (via GitHub)" <gi...@apache.org>]

arhoads opened a new issue, #346:
URL: https://github.com/apache/pulsar-client-cpp/issues/346

   ### Search before asking
   
   - [X] I searched in the [issues](https://github.com/apache/pulsar-client-cpp/issues) and found nothing similar.
   
   
   ### Version
   
   Commit 6f115e76da42ffbcf39a5fbe1df298e6d6635f4d
   
   ### Minimal reproduce step
   
   `ClientConnection::close` makes the function call `pool_.remove(logicalAddress_, this);`. LogicalAddress is no longer the correct key in the connection pool and instead should be logical address + "- {keySuffix}". The key isn't found in the pool map and a crash happens because the find result isn't validated.
   
   ### What did you expect to see?
   
   No crash
   
   ### What did you see instead?
   
   Crash
   
   ### Anything else?
   
   _No response_
   
   ### Are you willing to submit a PR?
   
   - [X] I'm willing to submit a PR!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] [Bug] ConnectionPool::remove Crash [pulsar-client-cpp]

["BewareMyPower (via GitHub)" <gi...@apache.org>]

BewareMyPower closed issue #346: [Bug] ConnectionPool::remove Crash
URL: https://github.com/apache/pulsar-client-cpp/issues/346


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org