Skip to content

Commit d35e292

Browse files
InterLinked1Friendly Automation
authored and
Friendly Automation
committed
res_rtp_asterisk: Fix typo in flag test/set
The code currently checks to see if an RFC3389 warning flag is set, except if it is, it merely sets the flag again, the logic of which doesn't make any sense. This adjusts the if comparison to check if the flag has NOT been set, and if so, emit a notice log event and set the flag so that future frames do not cause an event to be logged. ASTERISK-29856 #close Change-Id: Ib7098c947c63537d087a03b4646199fbb963f8e1
1 parent 97ace6b commit d35e292

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

Diff for: res/res_rtp_asterisk.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -5757,7 +5757,7 @@ static struct ast_frame *process_cn_rfc3389(struct ast_rtp_instance *instance, u
57575757
ast_format_get_name(rtp->lastrxformat), len);
57585758
}
57595759

5760-
if (ast_test_flag(rtp, FLAG_3389_WARNING)) {
5760+
if (!ast_test_flag(rtp, FLAG_3389_WARNING)) {
57615761
struct ast_sockaddr remote_address = { {0,} };
57625762

57635763
ast_rtp_instance_get_remote_address(instance, &remote_address);

0 commit comments

Comments
 (0)