[Asterisk-code-review] If rtupdate=no do not verify sipregs/peers table has updatab... (asterisk[master])

Jaco Kroon asteriskteam at digium.com
Sun Apr 17 15:38:40 CDT 2016


Jaco Kroon has uploaded a new change for review.

  https://gerrit.asterisk.org/2631

Change subject: If rtupdate=no do not verify sipregs/peers table has updatable fields.
......................................................................

If rtupdate=no do not verify sipregs/peers table has updatable fields.

Change-Id: Iaa2c53037b93daccc7e7333c40d61861847b856d
---
M channels/chan_sip.c
1 file changed, 13 insertions(+), 11 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/31/2631/1

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index 2a6dd77..e439cf9 100644
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -35267,17 +35267,19 @@
 	/* And start the monitor for the first time */
 	restart_monitor();
 
-	ast_realtime_require_field(ast_check_realtime("sipregs") ? "sipregs" : "sippeers",
-		"name", RQ_CHAR, 10,
-		"ipaddr", RQ_CHAR, INET6_ADDRSTRLEN - 1,
-		"port", RQ_UINTEGER2, 5,
-		"regseconds", RQ_INTEGER4, 11,
-		"defaultuser", RQ_CHAR, 10,
-		"fullcontact", RQ_CHAR, 35,
-		"regserver", RQ_CHAR, 20,
-		"useragent", RQ_CHAR, 20,
-		"lastms", RQ_INTEGER4, 11,
-		SENTINEL);
+	if (sip_cfg.peer_rtpupdate) {
+		ast_realtime_require_field(ast_check_realtime("sipregs") ? "sipregs" : "sippeers",
+			"name", RQ_CHAR, 10,
+			"ipaddr", RQ_CHAR, INET6_ADDRSTRLEN - 1,
+			"port", RQ_UINTEGER2, 5,
+			"regseconds", RQ_INTEGER4, 11,
+			"defaultuser", RQ_CHAR, 10,
+			"fullcontact", RQ_CHAR, 35,
+			"regserver", RQ_CHAR, 20,
+			"useragent", RQ_CHAR, 20,
+			"lastms", RQ_INTEGER4, 11,
+			SENTINEL);
+	}
 
 
 	sip_register_tests();

-- 
To view, visit https://gerrit.asterisk.org/2631
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iaa2c53037b93daccc7e7333c40d61861847b856d
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Jaco Kroon <jaco at uls.co.za>



More information about the asterisk-code-review mailing list