Module: kamailio
Branch: master
Commit: 77e7494d29d891c44a49377a462b0c82e6769f1e
URL: https://github.com/kamailio/kamailio/commit/77e7494d29d891c44a49377a462b0c8…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2024-11-26T10:46:10+01:00
modules: readme files regenerated - dispatcher ... [skip ci]
---
Modified: src/modules/dispatcher/README
---
Diff: https://github.com/kamailio/kamailio/commit/77e7494d29d891c44a49377a462b0c8…
Patch: https://github.com/kamailio/kamailio/commit/77e7494d29d891c44a49377a462b0c8…
---
diff --git a/src/modules/dispatcher/README b/src/modules/dispatcher/README
index ef5c56f6df6..44439a450d7 100644
--- a/src/modules/dispatcher/README
+++ b/src/modules/dispatcher/README
@@ -1678,6 +1678,11 @@ onreply_route {
The parameters can include variables.
+ The returns 1 (cfg-true) when the oc attrbutes are udpated for a
+ destination record; -1 (cfg-false) when there is an internal error; -2
+ (cfg-false) when no destination is updated (e.g., not found or the
+ ocseq condition fails).
+
This function can be used from ANY_ROUTE.
Example 1.61. ds_oc_set_attrs() usage
Module: kamailio
Branch: master
Commit: 76f2d91dc00fa33368d4df0d5a25a2dbbbc9f8c7
URL: https://github.com/kamailio/kamailio/commit/76f2d91dc00fa33368d4df0d5a25a2d…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2024-11-26T10:39:35+01:00
dispatcher: updated docs for ds_oc_set_attrs()
---
Modified: src/modules/dispatcher/doc/dispatcher_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/76f2d91dc00fa33368d4df0d5a25a2d…
Patch: https://github.com/kamailio/kamailio/commit/76f2d91dc00fa33368d4df0d5a25a2d…
---
diff --git a/src/modules/dispatcher/doc/dispatcher_admin.xml b/src/modules/dispatcher/doc/dispatcher_admin.xml
index 5e7cb95953c..f311be2f1ff 100644
--- a/src/modules/dispatcher/doc/dispatcher_admin.xml
+++ b/src/modules/dispatcher/doc/dispatcher_admin.xml
@@ -2073,6 +2073,10 @@ onreply_route {
</listitem>
</itemizedlist>
<para>The parameters can include variables.</para>
+ <para>The returns 1 (cfg-true) when the oc attrbutes are udpated for a
+ destination record; -1 (cfg-false) when there is an internal error;
+ -2 (cfg-false) when no destination is updated (e.g., not found or the
+ ocseq condition fails).</para>
<para>
This function can be used from ANY_ROUTE.
</para>
<!-- 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
- [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)
- [ ] 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 -->
Various fixes for issues discovered with the Coverity static analysis tooling.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/4030
-- Commit Summary --
* cdp: fixes for various issues found with coverity
-- File Changes --
M src/modules/cdp/authstatemachine.c (2)
M src/modules/cdp/common.c (8)
M src/modules/cdp/diameter_peer.c (6)
M src/modules/cdp/receiver.c (4)
M src/modules/cdp/session.c (12)
M src/modules/cdp/session.h (13)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/4030.patchhttps://github.com/kamailio/kamailio/pull/4030.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4030
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/4030(a)github.com>
Module: kamailio
Branch: master
Commit: 1ceeba7364b745b6ca2840e830d006a37460d10e
URL: https://github.com/kamailio/kamailio/commit/1ceeba7364b745b6ca2840e830d006a…
Author: Dragos Vingarzan <vingarzan(a)gmail.com>
Committer: Dragos Vingarzan <dragos(a)neatpath.net>
Date: 2024-11-25T13:33:40+01:00
cdp: fixed issues discovered during static code analysis
This fixes:
- 4-bytes (typically uint32_t) used for time_t storage (Y2K38)
- print of time_t with %d instead of %ld
- reuse of same loop index variable in a sub-loop
- use of random(), when kam_rand() could theoretically be better
- copy-paste mistake in sp2/sp parameter, where sp2 was meant and sp is probably NULL
---
Modified: src/modules/cdp/authstatemachine.c
Modified: src/modules/cdp/common.c
Modified: src/modules/cdp/diameter_peer.c
Modified: src/modules/cdp/receiver.c
Modified: src/modules/cdp/session.c
Modified: src/modules/cdp/session.h
---
Diff: https://github.com/kamailio/kamailio/commit/1ceeba7364b745b6ca2840e830d006a…
Patch: https://github.com/kamailio/kamailio/commit/1ceeba7364b745b6ca2840e830d006a…