diff options
author | Richard Mudgett <rmudgett@digium.com> | 2015-07-14 14:36:42 -0500 |
---|---|---|
committer | Richard Mudgett <rmudgett@digium.com> | 2015-07-15 19:30:13 -0500 |
commit | c782320c68633c9b1b805affaec1bfe604370d7f (patch) | |
tree | 225b6106f979f7065a966be402cd473c291af352 | |
parent | 5086bdacfb921ba3edbe49a1177163f5be846549 (diff) |
res_parking: Fix crash if ATTENDEDTRANSFER set empty before Park.
setup_park_common_datastore() was assuming that a non-NULL string returned
for the ATTENDEDTRANSFER and BLINDTRANSFER channel variables are not empty
strings. Things got crashy as a result.
* Made setup_park_common_datastore() treat the channel variable values the
same whether they are NULL or empty for ATTENDEDTRANSFER and
BLINDTRANSFER.
ASTERISK-25254 #close
Reported by: Richard Mudgett
Change-Id: I9a9c174b33f354f35f82cc6b7cea8303adbaf9c2
-rw-r--r-- | res/parking/parking_applications.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/res/parking/parking_applications.c b/res/parking/parking_applications.c index 22ae9c448..9446f2fa9 100644 --- a/res/parking/parking_applications.c +++ b/res/parking/parking_applications.c @@ -349,8 +349,12 @@ static int setup_park_common_datastore(struct ast_channel *parkee, const char *p attended_transfer = pbx_builtin_getvar_helper(parkee, "ATTENDEDTRANSFER"); blind_transfer = pbx_builtin_getvar_helper(parkee, "BLINDTRANSFER"); - if (attended_transfer || blind_transfer) { - parker_dial_string = ast_strdupa(S_OR(attended_transfer, blind_transfer)); + if (!ast_strlen_zero(attended_transfer)) { + parker_dial_string = ast_strdupa(attended_transfer); + } else if (!ast_strlen_zero(blind_transfer)) { + parker_dial_string = ast_strdupa(blind_transfer); + /* Ensure that attended_transfer is NULL and not an empty string. */ + attended_transfer = NULL; } ast_channel_unlock(parkee); |