Skip to content
Snippets Groups Projects
  • Pirmin Walthert's avatar
    e0785580
    bridge_channel.c: Fix Deadlock when using Local channels and fax gateway · e0785580
    Pirmin Walthert authored
    ast_indicate is invoked with the bridge locked. As ast_indicate locks the
    other end of the bridge as well this can lead to a deadlock in some situations.
    (Especially when a different thread does the same in the reverse order).
    This patch calls ast_indicate after unlocking the bridge which fixes the
    deadlock. Calling ast_indicate with these parameters without locking the
    bridge should be safe as this is done at different places without a
    bridge lock.
    
    ASTERISK-27094 #close
    Reported-by: David Brillert
    
    Change-Id: I5f86c1e2ce75b9929a36ab589b18c450e62ea35f
    e0785580
    History
    bridge_channel.c: Fix Deadlock when using Local channels and fax gateway
    Pirmin Walthert authored
    ast_indicate is invoked with the bridge locked. As ast_indicate locks the
    other end of the bridge as well this can lead to a deadlock in some situations.
    (Especially when a different thread does the same in the reverse order).
    This patch calls ast_indicate after unlocking the bridge which fixes the
    deadlock. Calling ast_indicate with these parameters without locking the
    bridge should be safe as this is done at different places without a
    bridge lock.
    
    ASTERISK-27094 #close
    Reported-by: David Brillert
    
    Change-Id: I5f86c1e2ce75b9929a36ab589b18c450e62ea35f