Hi everyone,
We have seen some crashes in kamailio version 5.3 which happens in some
modules who are getting the request for a reply from the transaction by the
tm module export and try to parse extra header in this request by the
parser functions. This seems to create a mix of shm with pkg which lead to
a crash in retransmission. My question is, is there a way to make sure that
there is no mix of shm and pkg or is the parsing on the message from the
transaction not a planned way of usage?
Best regards
Rick
Module: kamailio
Branch: master
Commit: 17e757c91e6e96f6711d9251979d81a792aaef08
URL: https://github.com/kamailio/kamailio/commit/17e757c91e6e96f6711d9251979d81a…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2021-04-21T10:55:18+02:00
dmq: docs for server_socket address
---
Modified: src/modules/dmq/doc/dmq_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/17e757c91e6e96f6711d9251979d81a…
Patch: https://github.com/kamailio/kamailio/commit/17e757c91e6e96f6711d9251979d81a…
---
diff --git a/src/modules/dmq/doc/dmq_admin.xml b/src/modules/dmq/doc/dmq_admin.xml
index 20a2e647b3..0e749e901e 100644
--- a/src/modules/dmq/doc/dmq_admin.xml
+++ b/src/modules/dmq/doc/dmq_admin.xml
@@ -121,7 +121,12 @@ sip:192.168.40.17:5060;status=active
<section id="dmq.p.server_address">
<title><varname>server_address</varname>(str)</title>
<para>
- The local server address. This is the interface over which the DMQ engine will send/receive messages.
+ The local server address. This is the addres with which the local DMQ
+ node joins the servers group to send/receive messages.
+ </para>
+ <para>
+ Note: if server_socket is not set, then this address has to match a
+ local socket, which will be used to send/receive messages.
</para>
<para>
<emphasis>Default value is <quote>NULL</quote>.</emphasis>
@@ -134,6 +139,28 @@ modparam("dmq", "server_address", "sip:10.0.0.20:5060")
...
modparam("dmq", "server_address", "sip:10.0.0.20:5061;transport=tls")
...
+</programlisting>
+ </example>
+ </section>
+ <section id="dmq.p.server_socket">
+ <title><varname>server_socket</varname>(str)</title>
+ <para>
+ The local listen address. This is the interface over which the DMQ
+ engine will send/receive messages.
+ </para>
+ <para>
+ Note: if this parameter is not set, the socket to send/receice
+ messages is built from server_address parameter.
+ </para>
+ <para>
+ <emphasis>Default value is <quote>NULL</quote>.</emphasis>
+ </para>
+ <example>
+ <title>Set <varname>server_socket</varname> parameter</title>
+ <programlisting format="linespecific">
+...
+modparam("dmq", "server_socket", "udp:10.0.0.20:5060")
+...
</programlisting>
</example>
</section>
<!-- 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 -->
- [X] Commit message has the format required by CONTRIBUTING guide
- [X] Commits are split per component (core, individual modules, libs, utils, ...)
- [X] Each component has a single commit (if not, squash them into one commit)
- [X] 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)
- [X] 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 -->
- [X] PR should be backported to stable branches
- [X] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
export functions of ims_charging and ims_diameter_server to kemi
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/2704
-- Commit Summary --
* Merge pull request #1 from kamailio/master
* ims_charging: export functions to kemi
* ims_diameter_server: export functions to kemi
-- File Changes --
M src/modules/ims_charging/ims_charging_mod.c (27)
M src/modules/ims_diameter_server/ims_diameter_server.c (21)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/2704.patchhttps://github.com/kamailio/kamailio/pull/2704.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/2704
- Added new sms pdu data: originating & destination address flags;
SCTS(DELIVER)/VP(SUBMIT) absolute time format.
- decode_3gpp_sms(): added support for DELIVER type;
Added VP-AbsoluteTimeFormat for SUBMIT type.
- pv_sms_body(): added support for ucs2 coding.
- Extends of pv_get_sms() and pv_set_sms(): added support for
originating & destination address flags.
<!-- 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 -->
- [x] Commit message has the format required by CONTRIBUTING guide
- [x] Commits are split per component (core, individual modules, libs, utils, ...)
- [x] Each component has a single commit (if not, squash them into one commit)
- [x] 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
- [x] Small bug fix (non-breaking change which fixes an issue)
- [x] 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
- [x] 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/2702
-- Commit Summary --
* smsops: Imrovements in decode_3gpp_sms() and pv_sms_body()
-- File Changes --
M src/modules/smsops/smsops_impl.c (343)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/2702.patchhttps://github.com/kamailio/kamailio/pull/2702.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/2702
#### Pre-Submission Checklist
- [*] 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:
- [ ] PR should be backported to stable branches
- [x] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
PR upgraded GCC for RPM packaging.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/2711
-- Commit Summary --
* pkg/kamailio/obs: used more recent gcc for RHEL and CentOS dists
-- File Changes --
M pkg/kamailio/obs/kamailio.spec (32)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/2711.patchhttps://github.com/kamailio/kamailio/pull/2711.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/2711