channel: Fix crash in suppress API.

There exists an inconsistency with framehook usage
such that it is only on reads that the frame should
be freed, not on writes as well.

ASTERISK-29071

Change-Id: I5ef918ebe4debac8a469e8d43bf9d6b673e8e472

diff --git a/main/channel.c b/main/channel.c
index 3d6fae64d335afc9be2c46de71a0d605bdce2044..e92eaf8bb29a0dec6ab844a50e67e43e84e8fbe0 100644 (file)
--- a/main/channel.c
+++ b/main/channel.c
@@ -10845,7 +10845,9 @@ static struct ast_frame *suppress_framehook_event_cb(struct ast_channel *chan, s
        if (suppress_frame) {
                switch (frame->frametype) {
                case AST_FRAME_VOICE:
-                       ast_frfree(frame);
+                       if (event == AST_FRAMEHOOK_EVENT_READ) {
+                               ast_frfree(frame);
+                       }
                        frame = &ast_null_frame;
                        break;
                default: