<!-- 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 -->
- [V] Commit message has the format required by CONTRIBUTING guide
- [V] Commits are split per component (core, individual modules, libs, utils, ...)
- [V] Each component has a single commit (if not, squash them into one commit)
- [V] 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)
- [V] 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 -->
- [V] PR should be backported to stable branches
- [V] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
I modified the imc module as follows:
1. propagate content-type header
2. add the option to modify member role
3. added support to db_mode. all changes are synch to db in case db_mode is set to 2.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3291
-- Commit Summary --
* imc: add support to modify member role
* imc: call handle_modify
* imc: db_mode support
* Merge branch 'kamailio:master' into master
* imc: destroy db only if db_mode is not 0
* Merge remote-tracking branch 'joeygo/master'
* imc: propagate content-type header
* Merge branch 'kamailio:master' into master
-- File Changes --
M src/modules/imc/doc/imc.xml (7)
M src/modules/imc/doc/imc_admin.xml (35)
M src/modules/imc/imc.c (326)
M src/modules/imc/imc.h (1)
M src/modules/imc/imc_cmd.c (260)
M src/modules/imc/imc_cmd.h (9)
M src/modules/imc/imc_mng.c (405)
M src/modules/imc/imc_mng.h (28)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3291.patchhttps://github.com/kamailio/kamailio/pull/3291.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3291
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3291(a)github.com>
- if t_suspend and t_continue are used for replies and in the called route in t_continue the body of the sip message is parsed the pkg memory of the parsed body is leaked
<!-- 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)
- [ ] 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/3294
-- Commit Summary --
* tm: cleanup parsed body at the end t_continue_helper
-- File Changes --
M src/modules/tm/t_suspend.c (5)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3294.patchhttps://github.com/kamailio/kamailio/pull/3294.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3294
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3294(a)github.com>
Module: kamailio
Branch: master
Commit: f409a8c9f6fde33eaf8b428887e219b56740b0a5
URL: https://github.com/kamailio/kamailio/commit/f409a8c9f6fde33eaf8b428887e219b…
Author: Rick Barenthin <rick(a)ng-voice.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2022-12-07T14:14:42+01:00
tm: cleanup parsed body at the end t_continue_helper
- if t_suspend and t_continue are used for replies
and in the called route in t_continue the body of
the sip message is parsed the pkg memory of the
parsed body is leaked
---
Modified: src/modules/tm/t_suspend.c
---
Diff: https://github.com/kamailio/kamailio/commit/f409a8c9f6fde33eaf8b428887e219b…
Patch: https://github.com/kamailio/kamailio/commit/f409a8c9f6fde33eaf8b428887e219b…
---
diff --git a/src/modules/tm/t_suspend.c b/src/modules/tm/t_suspend.c
index f0255d6647..4e25c3c89b 100644
--- a/src/modules/tm/t_suspend.c
+++ b/src/modules/tm/t_suspend.c
@@ -554,6 +554,11 @@ static int t_continue_helper(unsigned int hash_index, unsigned int label,
hdr = hdr->next;
}
}
+
+ /* trigger free of body */
+ if(brpl->body && brpl->body->free) {
+ brpl->body->free(&brpl->body);
+ }
sip_msg_free(brpl);
}
}