The astman_get_header() never returns NULL so the check by the code for
NULL would never fail.
(closes issue ASTERISK-16974)
Reported by: Nuno Borges
Patches:
0018325.patch (license #6116) patch uploaded by Nuno Borges (modified)
........
Merged revisions 354835 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........
Merged revisions 354836 from http://svn.asterisk.org/svn/asterisk/branches/10
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@354837
65c4cc65-6c06-0410-ace0-
fbb531ad65f3
ast_set_flag(chan2, AST_FLAG_BRIDGE_HANGUP_DONT); /* don't let the after-bridge code run the h-exten */
ast_channel_unlock(chan2);
}
- if (context2) {
+ if (!ast_strlen_zero(context2)) {
res = ast_async_goto(chan2, context2, exten2, pi2);
} else {
res = ast_async_goto(chan2, context, exten, pi);
astman_send_error(s, m, "Redirect failed");
}
- if (chan) {
- chan = ast_channel_unref(chan);
- }
-
+ chan = ast_channel_unref(chan);
if (chan2) {
chan2 = ast_channel_unref(chan2);
}