diff options
author | Richard Mudgett <rmudgett@digium.com> | 2016-12-21 17:54:42 -0600 |
---|---|---|
committer | Richard Mudgett <rmudgett@digium.com> | 2016-12-22 12:16:20 -0600 |
commit | 0aa5db4b38e80a059c3cd2c1d70ee9c509c0f488 (patch) | |
tree | 8d65446885b5e3e088f1e39b52c45f1529620cfe /channels | |
parent | bcdd282adaae7d47c91172d4e38306fc3b3566c7 (diff) |
chan_rtp.c: Fix uninitialized memory crash.
unicast_rtp_request() could pass an uninitialized 'us' parameter to
ast_ouraddrfor(). If ast_ouraddrfor() returns an error then the 'us'
parameter may not get initialized. Thus when the code tries to save the
'us' parameter to the local address we could try to copy a ridiculous
sized memory buffer and segfault.
* Made pass an initialized 'us' parameter to ast_ouraddrfor() and abort
the UnicastRTP channel request if it fails.
ASTERISK-26672
Change-Id: I1ef7a7c09f4da4f15dcb6de660d2bcac5f2a95c0
Diffstat (limited to 'channels')
-rw-r--r-- | channels/chan_rtp.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/channels/chan_rtp.c b/channels/chan_rtp.c index f1f4f05b9..d671706b2 100644 --- a/channels/chan_rtp.c +++ b/channels/chan_rtp.c @@ -316,7 +316,12 @@ static struct ast_channel *unicast_rtp_request(const char *type, struct ast_form engine_name = S_COR(ast_test_flag(&opts, OPT_RTP_ENGINE), opt_args[OPT_ARG_RTP_ENGINE], "asterisk"); - ast_ouraddrfor(&address, &local_address); + ast_sockaddr_copy(&local_address, &address); + if (ast_ouraddrfor(&address, &local_address)) { + ast_log(LOG_ERROR, "Could not get our address for sending media to '%s'\n", + args.destination); + goto failure; + } instance = ast_rtp_instance_new(engine_name, NULL, &local_address, NULL); if (!instance) { ast_log(LOG_ERROR, |