sr-dev January 2018

sr-dev@lists.kamailio.org

35 participants
368 discussions

[kamailio/kamailio] json: add API for json_parse and json_extract_field (#1416)
by Emmanuel Schmidbauer 27 Jan '18

27 Jan '18

You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/1416 -- Commit Summary -- * json: add API -- File Changes -- A src/modules/json/api.h (57) M src/modules/json/json_mod.c (31) -- Patch Links -- https://github.com/kamailio/kamailio/pull/1416.patch https://github.com/kamailio/kamailio/pull/1416.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/1416

2 1

git:master:038c3f16: Merge pull request #1413 from armenb/ws_check_bounds_before_reading_mask
by GitHub 26 Jan '18

26 Jan '18

Module: kamailio Branch: master Commit: 038c3f16d9a8371f00b1b8e34d37d6509465a471 URL: https://github.com/kamailio/kamailio/commit/038c3f16d9a8371f00b1b8e34d37d65… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: GitHub <noreply(a)github.com> Date: 2018-01-26T08:50:54+01:00 Merge pull request #1413 from armenb/ws_check_bounds_before_reading_mask websocket: check bounds before reading mask --- Modified: src/modules/websocket/ws_frame.c --- Diff: https://github.com/kamailio/kamailio/commit/038c3f16d9a8371f00b1b8e34d37d65… Patch: https://github.com/kamailio/kamailio/commit/038c3f16d9a8371f00b1b8e34d37d65… --- diff --git a/src/modules/websocket/ws_frame.c b/src/modules/websocket/ws_frame.c index 2739ecbc11..2d7ca8ec5a 100644 --- a/src/modules/websocket/ws_frame.c +++ b/src/modules/websocket/ws_frame.c @@ -470,13 +470,6 @@ static int decode_and_validate_ws_frame(ws_frame_t *frame, } else mask_start = 2; - /* Decode mask */ - frame->masking_key[0] = (buf[mask_start + 0] & 0xff); - frame->masking_key[1] = (buf[mask_start + 1] & 0xff); - frame->masking_key[2] = (buf[mask_start + 2] & 0xff); - frame->masking_key[3] = (buf[mask_start + 3] & 0xff); - - /* Decode and unmask payload */ if((unsigned long long)len != (unsigned long long)frame->payload_len + mask_start + 4) { LM_WARN("message not complete frame size %u but received %u\n", @@ -492,7 +485,15 @@ static int decode_and_validate_ws_frame(ws_frame_t *frame, *err_text = str_status_message_too_big; return -1; } + /* Decode mask */ + frame->masking_key[0] = (buf[mask_start + 0] & 0xff); + frame->masking_key[1] = (buf[mask_start + 1] & 0xff); + frame->masking_key[2] = (buf[mask_start + 2] & 0xff); + frame->masking_key[3] = (buf[mask_start + 3] & 0xff); + frame->payload_data = &buf[mask_start + 4]; + + /* Decode and unmask payload */ for(i = 0; i < frame->payload_len; i++) { j = i % 4; frame->payload_data[i] = frame->payload_data[i] ^ frame->masking_key[j];

1 0

git:master:4fabe253: websocket: check bounds before reading mask
by Armen Babikyan 26 Jan '18

26 Jan '18

Module: kamailio Branch: master Commit: 4fabe253a1eb0f9b494521cfa98365523a93adcf URL: https://github.com/kamailio/kamailio/commit/4fabe253a1eb0f9b494521cfa983655… Author: Armen Babikyan <armen(a)firespotter.com> Committer: Armen Babikyan <armen(a)firespotter.com> Date: 2018-01-25T17:43:33-08:00 websocket: check bounds before reading mask --- Modified: src/modules/websocket/ws_frame.c --- Diff: https://github.com/kamailio/kamailio/commit/4fabe253a1eb0f9b494521cfa983655… Patch: https://github.com/kamailio/kamailio/commit/4fabe253a1eb0f9b494521cfa983655… --- diff --git a/src/modules/websocket/ws_frame.c b/src/modules/websocket/ws_frame.c index 9bc3268601..32a1f4bf6a 100644 --- a/src/modules/websocket/ws_frame.c +++ b/src/modules/websocket/ws_frame.c @@ -470,13 +470,6 @@ static int decode_and_validate_ws_frame(ws_frame_t *frame, } else mask_start = 2; - /* Decode mask */ - frame->masking_key[0] = (buf[mask_start + 0] & 0xff); - frame->masking_key[1] = (buf[mask_start + 1] & 0xff); - frame->masking_key[2] = (buf[mask_start + 2] & 0xff); - frame->masking_key[3] = (buf[mask_start + 3] & 0xff); - - /* Decode and unmask payload */ if((unsigned long long)len != (unsigned long long)frame->payload_len + mask_start + 4) { LM_WARN("message not complete frame size %u but received %u\n", @@ -492,7 +485,15 @@ static int decode_and_validate_ws_frame(ws_frame_t *frame, *err_text = str_status_message_too_big; return -1; } + /* Decode mask */ + frame->masking_key[0] = (buf[mask_start + 0] & 0xff); + frame->masking_key[1] = (buf[mask_start + 1] & 0xff); + frame->masking_key[2] = (buf[mask_start + 2] & 0xff); + frame->masking_key[3] = (buf[mask_start + 3] & 0xff); + frame->payload_data = &buf[mask_start + 4]; + + /* Decode and unmask payload */ for(i = 0; i < frame->payload_len; i++) { j = i % 4; frame->payload_data[i] = frame->payload_data[i] ^ frame->masking_key[j];

1 0

[kamailio/kamailio] websocket: check bounds before reading mask (#1413)
by Armen Babikyan 26 Jan '18

26 Jan '18

#### 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 - [ ] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/1413 -- Commit Summary -- * websocket: check bounds before reading mask -- File Changes -- M src/modules/websocket/ws_frame.c (15) -- Patch Links -- https://github.com/kamailio/kamailio/pull/1413.patch https://github.com/kamailio/kamailio/pull/1413.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/1413

2 2

git:master:9811e318: Merge pull request #1412 from armenb/ws_cid85650_fix
by GitHub 26 Jan '18

26 Jan '18

Module: kamailio Branch: master Commit: 9811e318359b3099c3b1d7dff3673b642f8ccd0a URL: https://github.com/kamailio/kamailio/commit/9811e318359b3099c3b1d7dff3673b6… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: GitHub <noreply(a)github.com> Date: 2018-01-26T08:50:16+01:00 Merge pull request #1412 from armenb/ws_cid85650_fix websocket: handle unrecognized subprotocol better --- Modified: src/modules/websocket/ws_frame.c --- Diff: https://github.com/kamailio/kamailio/commit/9811e318359b3099c3b1d7dff3673b6… Patch: https://github.com/kamailio/kamailio/commit/9811e318359b3099c3b1d7dff3673b6… --- diff --git a/src/modules/websocket/ws_frame.c b/src/modules/websocket/ws_frame.c index 9bc3268601..2739ecbc11 100644 --- a/src/modules/websocket/ws_frame.c +++ b/src/modules/websocket/ws_frame.c @@ -678,6 +678,9 @@ int ws_frame_receive(sr_event_param_t *evp) return -1; } + } else { + LM_ERR("Unrecognized WebSocket subprotocol: %u\n", frame.wsc->sub_protocol); + return -1; } case OPCODE_CLOSE:

1 0

git:master:91c38814: websocket: handle unrecognized subprotocol better
by Armen Babikyan 26 Jan '18

26 Jan '18

Module: kamailio Branch: master Commit: 91c388147a9328743582a92c6f1184141aa268fb URL: https://github.com/kamailio/kamailio/commit/91c388147a9328743582a92c6f11841… Author: Armen Babikyan <armen(a)firespotter.com> Committer: Armen Babikyan <armen(a)firespotter.com> Date: 2018-01-25T17:20:35-08:00 websocket: handle unrecognized subprotocol better --- Modified: src/modules/websocket/ws_frame.c --- Diff: https://github.com/kamailio/kamailio/commit/91c388147a9328743582a92c6f11841… Patch: https://github.com/kamailio/kamailio/commit/91c388147a9328743582a92c6f11841… --- diff --git a/src/modules/websocket/ws_frame.c b/src/modules/websocket/ws_frame.c index 9bc3268601..2739ecbc11 100644 --- a/src/modules/websocket/ws_frame.c +++ b/src/modules/websocket/ws_frame.c @@ -678,6 +678,9 @@ int ws_frame_receive(sr_event_param_t *evp) return -1; } + } else { + LM_ERR("Unrecognized WebSocket subprotocol: %u\n", frame.wsc->sub_protocol); + return -1; } case OPCODE_CLOSE:

1 0

[kamailio/kamailio] websocket: handle unrecognized subprotocol better (#1412)
by Armen Babikyan 26 Jan '18

26 Jan '18

#### 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 - [ ] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/1412 -- Commit Summary -- * websocket: handle unrecognized subprotocol better -- File Changes -- M src/modules/websocket/ws_frame.c (3) -- Patch Links -- https://github.com/kamailio/kamailio/pull/1412.patch https://github.com/kamailio/kamailio/pull/1412.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/1412

2 2

git:master:1caf97ee: usrloc: proper condition when using rm_expired_delay
by Daniel-Constantin Mierla 26 Jan '18

26 Jan '18

Module: kamailio Branch: master Commit: 1caf97eee90fa5711b60f5cb7bf3a21a0144d4b6 URL: https://github.com/kamailio/kamailio/commit/1caf97eee90fa5711b60f5cb7bf3a21… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2018-01-26T08:41:19+01:00 usrloc: proper condition when using rm_expired_delay --- Modified: src/modules/usrloc/udomain.c --- Diff: https://github.com/kamailio/kamailio/commit/1caf97eee90fa5711b60f5cb7bf3a21… Patch: https://github.com/kamailio/kamailio/commit/1caf97eee90fa5711b60f5cb7bf3a21… --- diff --git a/src/modules/usrloc/udomain.c b/src/modules/usrloc/udomain.c index a3d948ad0b..8f5bfe3061 100644 --- a/src/modules/usrloc/udomain.c +++ b/src/modules/usrloc/udomain.c @@ -902,7 +902,7 @@ int db_timer_udomain(udomain_t* _d) keys[0] = &expires_col; ops[0] = "<"; vals[0].nul = 0; - UL_DB_EXPIRES_SET(&vals[0], act_time + 1 + ul_rm_expired_delay); + UL_DB_EXPIRES_SET(&vals[0], act_time + 1 - ul_rm_expired_delay); keys[1] = &expires_col; ops[1] = OP_NEQ;

1 0

git:master:f3064c68: modules: readme files regenerated - presence ... [skip ci]
by Kamailio Dev 25 Jan '18

25 Jan '18

Module: kamailio Branch: master Commit: f3064c6825840868ccfd4340d9bb6533e5842f67 URL: https://github.com/kamailio/kamailio/commit/f3064c6825840868ccfd4340d9bb653… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2018-01-25T19:46:34+01:00 modules: readme files regenerated - presence ... [skip ci] --- Modified: src/modules/presence/README --- Diff: https://github.com/kamailio/kamailio/commit/f3064c6825840868ccfd4340d9bb653… Patch: https://github.com/kamailio/kamailio/commit/f3064c6825840868ccfd4340d9bb653…

1 0

git:master:67bc67c5: Merge pull request #1402 from kamailio/cchance/presence_dmq
by GitHub 25 Jan '18

25 Jan '18

Module: kamailio Branch: master Commit: 67bc67c5487f29b6771d07ddd9e45cad5da46d43 URL: https://github.com/kamailio/kamailio/commit/67bc67c5487f29b6771d07ddd9e45ca… Author: Charles Chance <charles.chance(a)sipcentric.com> Committer: GitHub <noreply(a)github.com> Date: 2018-01-25T18:39:28Z Merge pull request #1402 from kamailio/cchance/presence_dmq presence: dmq integration --- Added: src/modules/presence/presence_dmq.c Added: src/modules/presence/presence_dmq.h Modified: src/lib/srdb1/schema/pr_presentity.xml Modified: src/modules/dmq/dmq.c Modified: src/modules/dmq/dmqnode.c Modified: src/modules/dmq/dmqnode.h Modified: src/modules/presence/Makefile Modified: src/modules/presence/doc/presence_admin.xml Modified: src/modules/presence/notify.c Modified: src/modules/presence/notify.h Modified: src/modules/presence/presence.c Modified: src/modules/presence/presence.h Modified: src/modules/presence/presentity.c Modified: src/modules/presence/presentity.h Modified: src/modules/presence/publish.c Modified: utils/kamctl/db_berkeley/kamailio/presentity Modified: utils/kamctl/db_berkeley/kamailio/version Modified: utils/kamctl/db_sqlite/presence-create.sql Modified: utils/kamctl/dbtext/kamailio/presentity Modified: utils/kamctl/dbtext/kamailio/version Modified: utils/kamctl/mongodb/kamailio/presentity.json Modified: utils/kamctl/mongodb/kamailio/version-create.mongo Modified: utils/kamctl/mysql/presence-create.sql Modified: utils/kamctl/oracle/presence-create.sql Modified: utils/kamctl/postgres/presence-create.sql Modified: utils/kamctl/xhttp_pi/pi_framework.xml Modified: utils/kamctl/xhttp_pi/presence-mod Modified: utils/kamctl/xhttp_pi/presence-table --- Diff: https://github.com/kamailio/kamailio/commit/67bc67c5487f29b6771d07ddd9e45ca… Patch: https://github.com/kamailio/kamailio/commit/67bc67c5487f29b6771d07ddd9e45ca…

1 0

← Newer
1
...
10
11
12
13
14
15
16
...
37
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev January 2018