]> git.ipfire.org Git - thirdparty/asterisk.git/commit
res_pjsip: Strip spaces from items parsed from comma-separated lists 58/2358/4
authorGeorge Joseph <george.joseph@fairview5.com>
Sun, 6 Mar 2016 20:38:41 +0000 (13:38 -0700)
committerGeorge Joseph <george.joseph@fairview5.com>
Mon, 7 Mar 2016 19:15:58 +0000 (12:15 -0700)
commit530cff5f5f3be4d2f53e84a656f10f2f43638d1c
tree8c61022785505b44dc0e9a7c9af3adbce66b39cd
parent6e58f83d8d68045574bcb432c74f38183a559bab
res_pjsip:  Strip spaces from items parsed from comma-separated lists

Configurations like "aors = a, b, c" were either ignoring everything after "a"
or trying to look up " b".  Same for mailboxes,  ciphers, contacts and a few
others.

To fix, all the strsep(&copy, ",") calls have been wrapped in ast_strip.  To
facilitate this, ast_strip, ast_skip_blanks and ast_skip_nonblanks were
updated to handle null pointers.

In some cases, an ast_strlen_zero() test was added to skip consecutive commas.

There was also an attempt to ast_free an ast_strdupa'd string in
ast_sip_for_each_aor which was causing a SEGV.  I removed it.

Although this issue was reported for realtime, the issue was in the res_pjsip
modules so all config mechanisms were affected.

ASTERISK-25829 #close
Reported-by: Mateusz Kowalski
Change-Id: I0b22a2cf22a7c1c50d4ecacbfa540155bec0e7a2
12 files changed:
channels/pjsip/dialplan_functions.c
include/asterisk/strings.h
res/res_pjsip/config_transport.c
res/res_pjsip/location.c
res/res_pjsip/pjsip_configuration.c
res/res_pjsip/pjsip_options.c
res/res_pjsip_endpoint_identifier_ip.c
res/res_pjsip_mwi.c
res/res_pjsip_notify.c
res/res_pjsip_path.c
res/res_pjsip_pubsub.c
res/res_pjsip_registrar.c