[Asterisk-code-review] main/file.c: Don't throw error on flash event. (asterisk[18])

Joshua Colp asteriskteam at digium.com
Mon May 17 08:53:50 CDT 2021


Joshua Colp has submitted this change. ( https://gerrit.asterisk.org/c/asterisk/+/15878 )

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; Approved for Submit
  George Joseph: Looks good to me, approved



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/+/15878
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 18
Gerrit-Change-Id: I4fa549bfb7ba1894a4044de999ea124877422fbc
Gerrit-Change-Number: 15878
Gerrit-PatchSet: 1
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/11fea7b5/attachment.html>


More information about the asterisk-code-review mailing list