Re: Crash in subscription destroy: error in mamaSubscription_getSubjectContext
Frank Quinn <fquinn@...>
Thanks Yury,toggle quoted messageShow quoted text
Raised https://github.com/OpenMAMA/OpenMAMA/issues/300 so we don't forget to get this sorted.
From: Yury Batrakov [mailto:firstname.lastname@example.org]
Sent: 26 June 2017 09:31
To: openmama-dev <email@example.com>; openmama-users <firstname.lastname@example.org>
Cc: Frank Quinn <email@example.com>
Subject: Crash in subscription destroy: error in mamaSubscription_getSubjectContext
There is a copy-paste error in mamaSubscription_getSubjectContext function that leads to crash on subscription destroy in case if issue symbol differs that the symbol for subscription. The problem is in this line: mama/c_cpp/src/c/subscription.c:1242 (link to GitHub: https://github.com/OpenMAMA/OpenMAMA/blob/master/mama/c_cpp/src/c/subscription.c#L1242):
entBridge->createSubscription (entBridge, &(self->mSubjectContext));
The last argument to createSubscription should be context, not &(self->mSubjectContext).
Please could you make appropriate change in the code?
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.
The information contained in this message may be privileged and confidential and protected from disclosure. If the reader of this message is not the intended recipient, you are hereby notified that any dissemination, distribution or copying of this communication is strictly prohibited. If you have received this communication in error, please notify us immediately by replying to this message and deleting it from your computer. Thank you. Vela Trading Technologies LLC