Argh... I suppose third time's the charm.

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@94660 65c4cc65-6c06-0410-ace0-fbb531ad65f3

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index c04020a61252ac98ac2a2c0d83510ffa3a0a03a6..d77510ab2717987639b093a65ff23ed81f30844c 100644 (file)
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -2500,33 +2500,34 @@ static struct sip_peer *realtime_peer(const char *newpeername, struct sockaddr_i
        /* First check on peer name */
        if (newpeername) {
                var = ast_load_realtime("sippeers", "name", newpeername, "host", "dynamic", NULL);
-               if (!var && sin) {
+               if (!var && sin)
                        var = ast_load_realtime("sippeers", "name", newpeername, "host", ast_inet_ntoa(sin->sin_addr), NULL);
-                       if (!var) {
-                               var = ast_load_realtime("sippeers", "name", newpeername, NULL);
-                               /*!\note
-                                * If this one loaded something, then we need to ensure that the host
-                                * field matched.  The only reason why we can't have this as a criteria
-                                * is because we only have the IP address and the host field might be
-                                * set as a name (and the reverse PTR might not match).
-                                */
-                               if (var) {
-                                       for (tmp = var; tmp; tmp = tmp->next) {
-                                               if (!strcasecmp(var->name, "host")) {
-                                                       struct in_addr sin2 = { 0, };
-                                                       struct ast_dnsmgr_entry *dnsmgr = NULL;
-                                                       if ((ast_dnsmgr_lookup(tmp->value, &sin2, &dnsmgr) < 0) || (memcmp(&sin2, &sin->sin_addr, sizeof(sin2)) != 0)) {
-                                                               /* No match */
-                                                               ast_variables_destroy(var);
-                                                               var = NULL;
-                                                       }
-                                                       break;
+               if (!var) {
+                       var = ast_load_realtime("sippeers", "name", newpeername, NULL);
+                       /*!\note
+                        * If this one loaded something, then we need to ensure that the host
+                        * field matched.  The only reason why we can't have this as a criteria
+                        * is because we only have the IP address and the host field might be
+                        * set as a name (and the reverse PTR might not match).
+                        */
+                       if (var) {
+                               for (tmp = var; tmp; tmp = tmp->next) {
+                                       if (!strcasecmp(var->name, "host")) {
+                                               struct in_addr sin2 = { 0, };
+                                               struct ast_dnsmgr_entry *dnsmgr = NULL;
+                                               if ((ast_dnsmgr_lookup(tmp->value, &sin2, &dnsmgr) < 0) || (memcmp(&sin2, &sin->sin_addr, sizeof(sin2)) != 0)) {
+                                                       /* No match */
+                                                       ast_variables_destroy(var);
+                                                       var = NULL;
                                                }
+                                               break;
                                        }
                                }
                        }
                }
-       } else if (sin) {       /* Then check on IP address */
+       }
+
+       if (!var && sin) {      /* Then check on IP address */
                iabuf = ast_inet_ntoa(sin->sin_addr);
                portnum = ntohs(sin->sin_port);
                sprintf(portstring, "%d", portnum);