[Asterisk-code-review] core/briding: correct crash case when bridge relay an empty ... (asterisk[13])

[Emmanuel BUU]

Hello Richard Mudgett, Kevin Harwell, Jenkins2, Corey Farrell, Joshua Colp, 

I'd like you to reexamine a change. Please visit

    https://gerrit.asterisk.org/10269

to look at the new patch set (#2).

Change subject: core/briding: correct crash case when bridge relay an empty text frame
......................................................................

core/briding: correct crash case when bridge relay an empty text frame

If a channel creates a AST_TEXT_FRAME with datalen == 0 and an invalid
dataptr, it causes a crash. The proposed fix is not to call sendtext
on such frames. It is not entirely transparent (it removes empty text
frames from the communication)

ASTERISK-28076
Reported by: Emmanuel BUU
Tested by: Emmanuel BUU

Change-Id: Ib882dd028598f13c4c233edbfdd7e54ad44a68e9
---
M main/bridge_channel.c
M main/frame.c
2 files changed, 3 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/69/10269/2
-- 
To view, visit https://gerrit.asterisk.org/10269
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib882dd028598f13c4c233edbfdd7e54ad44a68e9
Gerrit-Change-Number: 10269
Gerrit-PatchSet: 2
Gerrit-Owner: Emmanuel BUU <emmanuel.buu at ives.fr>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2 (1000185)
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20181001/049f69fd/attachment.html>