[Asterisk-code-review] main/file.c: Don't throw error on flash event. (asterisk[master])
Friendly Automation
asteriskteam at digium.com
Mon May 17 09:26:52 CDT 2021
Friendly Automation has submitted this change. ( https://gerrit.asterisk.org/c/asterisk/+/15888 )
Change subject: main/file.c: Don't throw error on flash event.
......................................................................
main/file.c: Don't throw error on flash event.
AST_CONTROL_FLASH isn't accounted for in a switch statement in file.c
where it should be ignored. Adding this to the switch ensures a
warning isn't thrown on RFC2833 flash events, since nothing's amiss.
ASTERISK-29372
Change-Id: I4fa549bfb7ba1894a4044de999ea124877422fbc
---
M main/file.c
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Joshua Colp: Looks good to me, but someone else must approve
George Joseph: Looks good to me, approved
Friendly Automation: Approved for Submit
diff --git a/main/file.c b/main/file.c
index b426609..61f64d2 100644
--- a/main/file.c
+++ b/main/file.c
@@ -1702,6 +1702,7 @@
case AST_CONTROL_AOC:
case AST_CONTROL_UPDATE_RTP_PEER:
case AST_CONTROL_PVT_CAUSE_CODE:
+ case AST_CONTROL_FLASH:
case -1:
/* Unimportant */
break;
--
To view, visit https://gerrit.asterisk.org/c/asterisk/+/15888
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I4fa549bfb7ba1894a4044de999ea124877422fbc
Gerrit-Change-Number: 15888
Gerrit-PatchSet: 2
Gerrit-Owner: N A <mail at interlinked.x10host.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20210517/812f4962/attachment.html>
More information about the asterisk-code-review
mailing list