Re: openmama 6.2 crash with dqstrategy disabled for subscription with multiple initials
toggle quoted messageShow quoted text
This was part of Vela's DQ strategy changes - Aaron could you have a look please?
Hi openmama team,
I see that commit 6de60350f40b4d9b0f181b7e64e0d192798b9f97 accidentally removed one conditional expression from around current line 393 of listenermsgcallbacks.c
Now if group subscription, or in fact any allowing multiple initials, receives first initial, then
- with dqstrategy disabled listenerMsgCallback_processMsg will unconditionally call mamaSubscription_stopWaitForResponse
- with dqstrategy enabled dqstrategy plugin implementation would only call mamaSubscription_stopWaitForResponse if mamaSubscription_getAcceptMultipleInitials() returns false
The code that stops waiting for response destroys related inbox object. This leads to easy crash with group subscriptions and dqstrategy disabled.
Hope it will be not hard to restore the conditional for openmama-6.2.x and later.
This e-mail may contain confidential and/or privileged information. If you are not the intended recipient (or have received this e-mail in error) please notify the sender immediately and delete this e-mail. Any unauthorized copying, disclosure or distribution
of the material in this e-mail is strictly forbidden.
Please refer to https://www.db.com/disclosures for additional EU corporate and regulatory disclosures and to http://www.db.com/unitedkingdom/content/privacy.htm for information about privacy.
Join Openmamafirstname.lastname@example.org to automatically receive all group messages.