]> git.kaiwu.me - nginx.git/commitdiff
Removed a surplus condition from ngx_parse_inet6_url().
authorRuslan Ermilov <ru@nginx.com>
Wed, 11 May 2016 14:55:30 +0000 (17:55 +0300)
committerRuslan Ermilov <ru@nginx.com>
Wed, 11 May 2016 14:55:30 +0000 (17:55 +0300)
No functional changes, since the condition was always true.

src/core/ngx_inet.c

index d5b7cf96a1ff904ae4c62994b5da7c468c6a4118..2ed4a14979c30ca7aeaf553f1eaa450a9923568d 100644 (file)
@@ -843,52 +843,49 @@ ngx_parse_inet6_url(ngx_pool_t *pool, ngx_url_t *u)
         return NGX_ERROR;
     }
 
-    if (last - p) {
+    port = p + 1;
 
-        port = p + 1;
+    uri = ngx_strlchr(port, last, '/');
 
-        uri = ngx_strlchr(port, last, '/');
+    if (uri) {
+        if (u->listen || !u->uri_part) {
+            u->err = "invalid host";
+            return NGX_ERROR;
+        }
 
-        if (uri) {
-            if (u->listen || !u->uri_part) {
-                u->err = "invalid host";
-                return NGX_ERROR;
-            }
+        u->uri.len = last - uri;
+        u->uri.data = uri;
 
-            u->uri.len = last - uri;
-            u->uri.data = uri;
+        last = uri;
+    }
 
-            last = uri;
+    if (port < last) {
+        if (*port != ':') {
+            u->err = "invalid host";
+            return NGX_ERROR;
         }
 
-        if (port < last) {
-            if (*port != ':') {
-                u->err = "invalid host";
-                return NGX_ERROR;
-            }
-
-            port++;
+        port++;
 
-            len = last - port;
+        len = last - port;
 
-            n = ngx_atoi(port, len);
+        n = ngx_atoi(port, len);
 
-            if (n < 1 || n > 65535) {
-                u->err = "invalid port";
-                return NGX_ERROR;
-            }
+        if (n < 1 || n > 65535) {
+            u->err = "invalid port";
+            return NGX_ERROR;
+        }
 
-            u->port = (in_port_t) n;
-            sin6->sin6_port = htons((in_port_t) n);
+        u->port = (in_port_t) n;
+        sin6->sin6_port = htons((in_port_t) n);
 
-            u->port_text.len = len;
-            u->port_text.data = port;
+        u->port_text.len = len;
+        u->port_text.data = port;
 
-        } else {
-            u->no_port = 1;
-            u->port = u->default_port;
-            sin6->sin6_port = htons(u->default_port);
-        }
+    } else {
+        u->no_port = 1;
+        u->port = u->default_port;
+        sin6->sin6_port = htons(u->default_port);
     }
 
     len = p - host;