<div dir="ltr">Daniel,<div><br></div><div>Yes, that is an error.  I should have used t_release_transaction() instead.  I will correct it and resubmit.</div><div><br></div><div>My apologies.  I rebuilt the OS on this machine and I'm still trying to get some things together.</div><div>Bob</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 24, 2021 at 5:26 AM Daniel-Constantin Mierla <<a href="mailto:miconda@gmail.com">miconda@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello,<br>
<br>
compiling the latest version of Kamailio results in a warning for<br>
mohqueue module:<br>
<br>
CC (gcc) [M mohqueue.so]        mohq_funcs.o<br>
mohq_funcs.c:225:23: warning: incompatible pointer types passing<br>
'tm_cell_t *' (aka 'struct cell *') to parameter of type 'struct sip_msg *'<br>
      [-Wincompatible-pointer-types]<br>
  if (ptm->t_release (ptcell) < 0)<br>
<br>
I noticed you pushed some changes recently, likely the warning was<br>
introduced by them, not noticing it before. Anyhow, it seems something<br>
that should be reviewed and fixed, otherwise may result in a crash by<br>
passing incorrect structure pointer.<br>
<br>
Cheers,<br>
Daniel<br>
<br>
-- <br>
Daniel-Constantin Mierla -- <a href="http://www.asipto.com" rel="noreferrer" target="_blank">www.asipto.com</a><br>
<a href="http://www.twitter.com/miconda" rel="noreferrer" target="_blank">www.twitter.com/miconda</a> -- <a href="http://www.linkedin.com/in/miconda" rel="noreferrer" target="_blank">www.linkedin.com/in/miconda</a><br>
Funding: <a href="https://www.paypal.me/dcmierla" rel="noreferrer" target="_blank">https://www.paypal.me/dcmierla</a><br>
<br>
<br>
_______________________________________________<br>
Kamailio (SER) - Development Mailing List<br>
<a href="mailto:sr-dev@lists.kamailio.org" target="_blank">sr-dev@lists.kamailio.org</a><br>
<a href="https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev" rel="noreferrer" target="_blank">https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev</a><br>
</blockquote></div>