Skip to content
Snippets Groups Projects
Select Git revision
  • dev_fxs_no_wb
  • fix_transport_fallback_at_dns_failure
  • 16302-renew-registration
  • asterisk_rdkb
  • 14600-wenpeng
  • fix_fallback
  • 14666_fxs_no_wideband_codec
  • fix_srv_records
  • wenpeng-test-250121
  • wenpeng-registration-test
  • fix_deadlock_in_bridge_peer_functions
  • devel default protected
  • securesip_pjproject_part1
  • PJ_Example_Review
  • fix_re_register_authorization_header
  • fix_internal_calls_auth
  • fix_internal_calls
  • Secure_SIPtelephony_part1
  • SIPtelephony_DECT_syslog
  • syslog_develop_phase1
  • 22.0.0-pre1
  • 21.4.2
  • 20.9.2
  • 18.24.2
  • certified-20.7-cert2
  • certified-18.9-cert11
  • 21.4.1
  • 20.9.1
  • 18.24.1
  • 21.4.0
  • 20.9.0
  • 18.24.0
  • certified-20.7-cert1
  • certified-18.9-cert10
  • 21.4.0-rc1
  • 20.9.0-rc1
  • 18.24.0-rc1
  • 21.3.1
  • 20.8.1
  • 18.23.1
40 results
Compare
user avatar
bridge_channel_write_frame: Check for NULL channel
George Joseph authored 
There is a possibility, when bridge_channel_write_frame() is
called, that the bridge_channel->chan will be NULL.  The first
thing bridge_channel_write_frame() does though is call
ast_channel_is_multistream() which had no check for a NULL
channel and therefore caused a segfault. Since it's still
possible for bridge_channel_write_frame() to write the frame to
the other channels in the bridge, we don't want to bail before we
call ast_channel_is_multistream() but we can just skip the
multi-channel stuff.  So...

bridge_channel_write_frame() only calls ast_channel_is_multistream()
if bridge_channel->chan is not NULL.

As a safety measure, ast_channel_is_multistream() now returns
false if the supplied channel is NULL.

ASTERISK-29379
Reported-by: Vyrva Igor
Reported-by: Ross Beer

Change-Id: Idfe62dbea8c69813ecfd58e113a6620dc42352ce
88aec107
History
user avatar 88aec107
History
Name Last commit Last update
..
stdtime
.gitignore
Makefile
abstract_jb.c
acl.c
adsi.c
alaw.c
alertpipe.c
aoc.c
app.c
ast_expr2.c
ast_expr2.fl
ast_expr2.h
ast_expr2.y
ast_expr2f.c
asterisk.c
asterisk.dynamics
asterisk.exports.in
astfd.c
astmm.c
astobj2.c
astobj2_container.c
astobj2_container_private.h
astobj2_global.c
astobj2_hash.c
astobj2_private.h
astobj2_rbtree.c
audiohook.c
autochan.c
autoservice.c
backtrace.c
bridge.c
bridge_after.c
bridge_basic.c
bridge_channel.c
bridge_roles.c
bucket.c
buildinfo.c
callerid.c
ccss.c
cdr.c
cel.c
channel.c
channel_internal_api.c
chanvars.c
cli.c
codec.c
codec_builtin.c
config.c
config_options.c
conversions.c
core_local.c
core_unreal.c
crypt.c
cygload.c
data_buffer.c
datastore.c
db.c
devicestate.c
dial.c
dns.c
dns_core.c
dns_naptr.c
dns_query_set.c
dns_recurring.c
dns_srv.c
dns_system_resolver.c
dns_test.c
dns_tlsa.c
dns_txt.c
dnsmgr.c
dsp.c
ecdisa.h
endpoints.c
enum.c
event.c
features.c
features_config.c
features_config.h
file.c
fixedjitterbuf.c
fixedjitterbuf.h
format.c
format_cache.c
format_cap.c
format_compatibility.c
frame.c
framehook.c
fskmodem.c
fskmodem_float.c
fskmodem_int.c
global_datastores.c
hashtab.c
heap.c
http.c
image.c
indications.c
io.c
iostream.c
jitterbuf.c