[sr-dev] git:master:387de23a: rtpengine: do nothing in rtpengine_manage() when used in event_route[tm:branch-failure:...]

[Daniel-Constantin Mierla]

Module: kamailio
Branch: master
Commit: 387de23a5c9252ee8409d378ea452bfd52e416d4
URL: https://github.com/kamailio/kamailio/commit/387de23a5c9252ee8409d378ea452bfd52e416d4

Author: Daniel-Constantin Mierla <miconda at gmail.com>
Committer: Daniel-Constantin Mierla <miconda at gmail.com>
Date: 2020-02-23T12:20:17+01:00

rtpengine: do nothing in rtpengine_manage() when used in event_route[tm:branch-failure:...]

- avoid duplicating sdp operations

---

Modified: src/modules/rtpengine/rtpengine.c

---

Diff:  https://github.com/kamailio/kamailio/commit/387de23a5c9252ee8409d378ea452bfd52e416d4.diff
Patch: https://github.com/kamailio/kamailio/commit/387de23a5c9252ee8409d378ea452bfd52e416d4.patch

---

diff --git a/src/modules/rtpengine/rtpengine.c b/src/modules/rtpengine/rtpengine.c
index 4db1461f0d..c040c36444 100644
--- a/src/modules/rtpengine/rtpengine.c
+++ b/src/modules/rtpengine/rtpengine.c
@@ -3616,9 +3616,14 @@ rtpengine_manage(struct sip_msg *msg, const char *flags)
 	int nosdp;
 	tm_cell_t *t = NULL;
 
+	if(route_type==BRANCH_FAILURE_ROUTE) {
+		/* do nothing in branch failure event route
+		 * - delete done on transaction failure route */
+		return 1;
+	}
+
 	if (msg->cseq==NULL && ((parse_headers(msg, HDR_CSEQ_F, 0)==-1) ||
-	   (msg->cseq==NULL)))
-	{
+	   (msg->cseq==NULL))) {
 		LM_ERR("no CSEQ header\n");
 		return -1;
 	}
@@ -3626,7 +3631,7 @@ rtpengine_manage(struct sip_msg *msg, const char *flags)
 	method = get_cseq(msg)->method_id;
 
 	if (!(method==METHOD_INVITE || method==METHOD_ACK || method==METHOD_CANCEL
-	   || method==METHOD_BYE || method==METHOD_UPDATE))
+			|| method==METHOD_BYE || method==METHOD_UPDATE))
 		return -1;
 
 	if (method==METHOD_CANCEL || method==METHOD_BYE)