th_mask_callid_str & th_unmask_callid_str functions used for API call to encode/decode call-ID uses static array declaration for callid_mbuf was unable to free callid data lump after use and leads to memory leak. when these functions was used through API call for topos module memory leak bug as reported
qm_free(): BUG: bad pointer 0x7faec4a7xxxx (out of memory block!) called from core: core/data_lump.c: free_lump(470)
This two funtions are not used other than API.
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [ ] Commit message has the format required by CONTRIBUTING guide
- [ ] Commits are split per component (core, individual modules, libs, utils, ...)
- [ ] Each component has a single commit (if not, squash them into one commit)
- [ ] No commits to README files for modules (changes must be done to docbook files
in `doc/` subfolder, the README file is autogenerated)
#### Type Of Change
- [ ] Small bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds new functionality)
- [ ] Breaking change (fix or feature that would change existing functionality)
#### Checklist:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [ ] PR should be backported to stable branches
- [ ] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3456
-- Commit Summary --
* topoh: memory leak fix for API call
-- File Changes --
M src/modules/topoh/th_msg.c (36)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3456.patchhttps://github.com/kamailio/kamailio/pull/3456.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3456
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3456(a)github.com>
**incompatible-pointer-types**
```
make[3]: 'libsrdb1.so.1.0' is up to date.
LD (gcc) [M sipcapture.so] sipcapture.so
CC (gcc) [M siptrace.so] siptrace.o
siptrace.c: In function 'ki_sip_trace_msg':
siptrace.c:1234:74: warning: passing argument 4 of 'trace_send_hep_duplicate' from incompatible pointer type [-Wincompatible-pointer-types]
trace_send_hep_duplicate(vmsg, saddr, taddr, (duri && duri->len>0)?&duri:NULL,
^
In file included from siptrace.c:58:
siptrace_hep.h:30:34: note: expected 'struct dest_info *' but argument is of type 'str **' {aka 'struct _str **'}
str *body, str *from, str *to, struct dest_info *, str *correlation_id);
^~~~~~~~~~~~~~~~~~
CC (gcc) [M siptrace.so] siptrace_hep.o
CC (gcc) [M siptrace.so] siptrace_send.o
```
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/3457
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/3457(a)github.com>
Hello,
with a few public holidays coming soon around here, I am considering to
release 5.7.0 this week on Wednesday, May 17, 2023. Rather short notice,
but there were no major bug reports that need to be tracked.
The list of new features is being built in the wiki:
-
https://github.com/kamailio/kamailio-wiki/blob/main/docs/features/new-in-5.…
The other tutorials related to this major release should be ready by
then as well.
Should anyone think more time is needed, then it can be postponed, not a
problem at all.
Cheers,
Daniel
--
Daniel-Constantin Mierla -- www.asipto.comwww.twitter.com/miconda -- www.linkedin.com/in/miconda
Kamailio World Conference - June 5-7, 2023 - www.kamailioworld.com
Module: kamailio
Branch: 5.7
Commit: 422d469fb52c31074ff00eec445eff45fdfebb39
URL: https://github.com/kamailio/kamailio/commit/422d469fb52c31074ff00eec445eff4…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2023-05-16T18:16:35+02:00
modules: readme files regenerated - modules ... [skip ci]
---
Modified: src/modules/topos/README
---
Diff: https://github.com/kamailio/kamailio/commit/422d469fb52c31074ff00eec445eff4…
Patch: https://github.com/kamailio/kamailio/commit/422d469fb52c31074ff00eec445eff4…
---
diff --git a/src/modules/topos/README b/src/modules/topos/README
index 1f8e8c191dd..be21cb2ef51 100644
--- a/src/modules/topos/README
+++ b/src/modules/topos/README
@@ -171,6 +171,7 @@ Chapter 1. Admin Guide
requests).
* database module - to store the data for topology stripping and
restoring.
+ * topoh module - in case of Call-ID mask for Call-ID masking.
2.2. External Libraries or Applications
@@ -228,11 +229,11 @@ modparam("topos", "db_url", "dbdriver://username:password@dbhost/dbname")
3.3. mask_callid (int)
- Note: this functionality is not implemented yet - the parameter is
- present in order to be in pair with topoh module.
-
- Whether to replace or not the Call-ID with another unique id generated
- by Kamailio.
+ Whether to encode the Call-id: header. Some SIP extensions include the
+ Call-id in the SIP message payload or header, so it is safe to not
+ encode Call-id in such cases. Well-known extensions such as call
+ transfer or conference join will be added to work with encoded Call-id.
+ Topoh module should be loaded with use_mode paramerter set to 1
Default value is 0 (do not mask).
Module: kamailio
Branch: 5.7
Commit: 7e1f330e2dc916138aa9a899da49c452b9f59740
URL: https://github.com/kamailio/kamailio/commit/7e1f330e2dc916138aa9a899da49c45…
Author: harish <toharishs(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2023-05-16T18:03:54+02:00
Topos: Added CallID Mask Document for Topos
Added document for Call-ID Mask in Topos Module
(cherry picked from commit 14d4b2422c0fed27fecf01665197f9d0669aa24e)
---
Modified: src/modules/topos/doc/topos_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/7e1f330e2dc916138aa9a899da49c45…
Patch: https://github.com/kamailio/kamailio/commit/7e1f330e2dc916138aa9a899da49c45…
---
diff --git a/src/modules/topos/doc/topos_admin.xml b/src/modules/topos/doc/topos_admin.xml
index dcf61a0fed6..82e7c1e45eb 100644
--- a/src/modules/topos/doc/topos_admin.xml
+++ b/src/modules/topos/doc/topos_admin.xml
@@ -57,6 +57,12 @@
for topology stripping and restoring.
</para>
</listitem>
+ <listitem>
+ <para>
+ <emphasis>topoh module</emphasis> - in case of Call-ID mask
+ for Call-ID masking.
+ </para>
+ </listitem>
</itemizedlist>
</para>
</section>
@@ -130,14 +136,14 @@ modparam("topos", "db_url", "&exampledb;")
<section id="topos.p.mask_callid">
<title><varname>mask_callid</varname> (int)</title>
<para>
- Note: this functionality is not implemented yet - the
- parameter is present in order to be in pair with topoh
- module.
- </para>
- <para>
- Whether to replace or not the Call-ID with another
- unique id generated by &kamailio;.
+ Whether to encode the Call-id: header. Some SIP extensions include
+ the Call-id in the SIP message payload or header, so it is safe to
+ not encode Call-id in such cases. Well-known extensions such as
+ call transfer or conference join will be added to work with encoded
+ Call-id.
+ Topoh module should be loaded with use_mode paramerter set to 1
</para>
+
<para>
<emphasis>
Default value is 0 (do not mask).