<div dir="ltr"><p>Hi All,</p>
<p>Whilst attempting to track down the source of an issue introduced at some point since 4.2.1, I came across the following and would appreciate a second opinion.<br></p>
<p dir="ltr">In send_notify_request() function (/modules/presence/notify.c), the aux body processing function attached to the event is called:</p><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">/* call aux_body_processing if exists */<br>if(subs->event->aux_body_processing)<br>{<br><span style="white-space:pre-wrap">       </span>aux_body = subs->event->aux_body_processing(subs, notify_body);<br><span style="white-space:pre-wrap"> </span>if(aux_body) {<br><span style="white-space:pre-wrap">                </span>free_notify_body(notify_body, subs->event);<br><span style="white-space:pre-wrap">                </span>notify_body = aux_body;<br><span style="white-space:pre-wrap">       </span>}<br>}</blockquote><p dir="ltr"><br></p><p dir="ltr">However, by the time we reach this function, aux body processing will already have been completed (specifically in the publ_notify() or query_db_notify() functions of the same file).</p><p dir="ltr">If I remove the duplicate function call, the issue goes away - but I'm reluctant to submit a pull request which reverts an earlier commit, unless I can understand the reason for the change in the first place. Especially since it was made a year ago.</p><p>Is anyone able to explain what I'm missing?<br></p><p dir="ltr">Cheers,<br></p><p>Charles</p>
</div>

<br>
<div><font face="Helvetica, Arial, sans-serif"><font size="2"><b>** We're rebranding! Learn more <a href="http://www.sipcentric.com/2015/07/were-rebranding/" target="_blank">here</a> **</b></font></font></div><div><br></div><font face="Helvetica, Arial, sans-serif"><font size="2"><span style="font-size:10pt"><a href="http://www.sipcentric.com/" title="blocked::http://www.sipcentric.com/" target="_blank">www.sipcentric.com</a><br>
            <br>
            Follow us on twitter <a href="http://twitter.com/sipcentric" title="blocked::http://twitter.com/sipcentric" target="_blank">@sipcentric</a><br>
            <br>
            <font color="gray">Sipcentric Ltd.
                Company registered in England & Wales no. 7365592.</font> <font color="gray">Registered
                office: Faraday Wharf, Innovation Birmingham Campus, Holt Street, Birmingham Science Park, Birmingham B7 4BB.</font></span></font></font>