Module: kamailio
Branch: master
Commit: 8d69c0d68347198ccaea57a8ac4eeb040d19287a
URL: https://github.com/kamailio/kamailio/commit/8d69c0d68347198ccaea57a8ac4eeb0…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2023-03-31T18:24:24+02:00
tm: rename shortcuts to from/to/callid/cseq headers
- reflect better that they are with header names
- new fields to point to callid value and cseq number to facilitate use
of them directly without new parsing or printing with header names
---
Modified: src/modules/tm/h_table.c
Modified: src/modules/tm/h_table.h
Modified: src/modules/tm/t_cancel.c
Modified: src/modules/tm/t_fwd.c
Modified: src/modules/tm/t_lookup.c
Modified: src/modules/tm/t_msgbuilder.c
Modified: src/modules/tm/t_stats.c
Modified: src/modules/tm/uac.c
---
Diff: https://github.com/kamailio/kamailio/commit/8d69c0d68347198ccaea57a8ac4eeb0…
Patch: https://github.com/kamailio/kamailio/commit/8d69c0d68347198ccaea57a8ac4eeb0…
### Environment
- **OS**: Debian 11 (64-bit)
### Description
While attempting to start kamailio thu the `kamctl start` command, I had recieved a following output in the terminal:
**Note**: I had edited the PID_FILE variable and had set the value to: /run/kamailio.pid
```
-e database engine 'SQLITE' loaded
-e Control engine 'RPCFIFO' loaded
-e \E[37;33mINFO: Starting Kamailio :
-e \E[37;31mERROR: PID file /run/kamailio.pid does not exist -- Kamailio start failed
```
In my opinion, it's strange that the server complains about missing PID file, while attempting to run it and no other Kamailio instance is running
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/3410
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/3410(a)github.com>
<!-- 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 -->
- [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 -->
segfault
```
#0 0x0000ffffb9934840 in cfg_update_local (no_cbs=0) at ../../core/cfg/cfg_struct.h:366
366 cfg_child_cb = cfg_child_cb->next;
(gdb) print cfg_child_cb
$1 = (cfg_child_cb_t *) 0x0
```
example lua:
```
function ksr_route_nats()
KSR.tm.t_newtran()
local tindex = KSR.pv.gete("$T(id_index)")
local tlabel = KSR.pv.gete("$T(id_label)")
KSR.tmx.t_suspend()
local req = {
jsonrpc = "2.0", method = "route",
id = tindex .. ':' .. tlabel,
params = {
from = KSR.pv.gete("$fu"),
to = KSR.pv.gete("$tu"),
}
}
local rpc = cjson.encode(req)
KSR.info("rpc: " .. rpc .. "\n")
KSR.nats.publish("cn.xswitch.ctrl.route", rpc)
KSR.x.exit();
end
function handle_nats_response(data)
msg = cjson.decode(data)
if (not msg) or (not msg.id) then return end
KSR.info("========= msg.id: " .. msg.id .. "\n")
if not msg.result then
KSR.error("response has no result\n")
return
end
tindex, tlabel = msg.id:match("(.+):(.+)")
if tindex and tlabel then
KSR.pv.sets('$var(nats_response)', data)
KSR.tmx.t_continue(tindex, tlabel, 'ksr_nats_continue')
end
end
```
I'm not sure if it's the proper fix, but this pr fixed the seg. Thanks.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3396
-- Commit Summary --
* core: fix a seg while t_continue with nats module
-- File Changes --
M src/core/cfg/cfg_struct.h (2)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3396.patchhttps://github.com/kamailio/kamailio/pull/3396.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3396
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3396(a)github.com>
<!-- 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 -->
- [ ] 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 -->
In `tcpops` module, the method `tcp_get_conid()` was missing which may be needed to trace, track and log TCP connections, especially for UDP to TCP reply route in KEMI. The patch is tested to work with `master` and `5.6` and above branches.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3402
-- Commit Summary --
* tcpops: added tcp_get_conid() for kemi
-- File Changes --
M src/modules/tcpops/tcpops_mod.c (25)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3402.patchhttps://github.com/kamailio/kamailio/pull/3402.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3402
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3402(a)github.com>
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3375
-- Commit Summary --
* acc: typo
* acc_json: typos
* acc_radius: typo
* alias_db: typo
* app_java: typos
* app_jsdt: typos
* app_lua: typos
* app_lua_sr: typos
* app_mono: typos
* app_perl: typos
* app_python: typo
* app_ruby: typos
* app_sqlang: typos
* async: typos
* auth: typos
* auth_db: typos
* auth_diameter: typos
* auth_ephemeral: typos
* auth_identiy: typos
* auth_radius: typos
* avpops: typos
-- File Changes --
M src/modules/acc/doc/acc_admin.xml (2)
M src/modules/acc_json/acc_json_mod.c (2)
M src/modules/acc_json/doc/acc_json_admin.xml (2)
M src/modules/acc_radius/doc/acc_radius_admin.xml (2)
M src/modules/alias_db/doc/alias_db_admin.xml (2)
M src/modules/app_java/README-draft (12)
M src/modules/app_java/doc/app_java_admin.xml (4)
M src/modules/app_java/kamailio_java_folder/java-untested/siprouter_src/SipMsg.java (2)
M src/modules/app_java/kamailio_java_folder/java/siprouter_src/SipMsg.java (2)
M src/modules/app_jsdt/app_jsdt_mod.c (8)
M src/modules/app_jsdt/doc/app_jsdt_admin.xml (4)
M src/modules/app_jsdt/duktape.c (10)
M src/modules/app_lua/app_lua_mod.c (10)
M src/modules/app_lua/doc/app_lua_admin.xml (2)
M src/modules/app_lua_sr/doc/app_lua_sr_admin.xml (4)
M src/modules/app_mono/app_mono_mod.c (4)
M src/modules/app_perl/app_perl_mod.c (2)
M src/modules/app_perl/doc/app_perl_admin.xml (4)
M src/modules/app_perl/doc/app_perl_pod.xml (2)
M src/modules/app_perl/doc/app_perl_samples.xml (2)
M src/modules/app_perl/lib/perl/Kamailio/LDAPUtils/LDAPConf.pm (4)
M src/modules/app_python/doc/app_python_admin.xml (2)
M src/modules/app_ruby/app_ruby_mod.c (2)
M src/modules/app_ruby/doc/app_ruby_admin.xml (2)
M src/modules/app_sqlang/app_sqlang_mod.c (8)
M src/modules/app_sqlang/doc/app_sqlang_admin.xml (2)
M src/modules/app_sqlang/squirrel/sqstdlib/sqstdrex.cpp (2)
M src/modules/app_sqlang/squirrel/squirrel/sqvm.cpp (6)
M src/modules/async/doc/async_admin.xml (4)
M src/modules/auth/auth.xml (18)
M src/modules/auth/doc/auth_functions.xml (2)
M src/modules/auth/doc/auth_params.xml (14)
M src/modules/auth/nc.c (6)
M src/modules/auth/nid.h (4)
M src/modules/auth/ot_nonce.c (6)
M src/modules/auth_db/authorize.c (2)
M src/modules/auth_db/doc/auth_db_admin.xml (4)
M src/modules/auth_diameter/authorize.c (2)
M src/modules/auth_diameter/authorize.h (2)
M src/modules/auth_diameter/avp.c (2)
M src/modules/auth_diameter/defs.h (2)
M src/modules/auth_diameter/message.c (2)
M src/modules/auth_diameter/tcp_comm.c (2)
M src/modules/auth_ephemeral/auth_ephemeral_mod.c (2)
M src/modules/auth_ephemeral/doc/auth_ephemeral_admin.xml (2)
M src/modules/auth_identity/auth_hdrs.c (2)
M src/modules/auth_identity/auth_identity.c (10)
M src/modules/auth_identity/auth_tables.c (10)
M src/modules/auth_identity/doc/auth_identity_functions.xml (2)
M src/modules/auth_radius/doc/auth_radius_admin.xml (2)
M src/modules/auth_radius/sterman.c (2)
M src/modules/avpops/avpops.c (4)
M src/modules/avpops/doc/avpops_admin.xml (8)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3375.patchhttps://github.com/kamailio/kamailio/pull/3375.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3375
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3375(a)github.com>