commit e34ff16f67cd2a243726bc37441ba48ba3643e22
author Maxim Dounin <mdounin@mdounin.ru> Thu Mar 22 11:57:18 2012 +0000
committer Maxim Dounin <mdounin@mdounin.ru> Thu Mar 22 11:57:18 2012 +0000
tree f4801626bc3145ced65bb332175dece1647b658a
parent 8e6728ef13f3bc3ae9d674dcd59067869d2d7c24

Resolver: added missing sanity checking when creating name queries.

Found by Veracode.

src/core/ngx_resolver.c

diff --git a/src/core/ngx_resolver.c b/src/core/ngx_resolver.c
index 2e30471..02c484d 100644
--- a/src/core/ngx_resolver.c
+++ b/src/core/ngx_resolver.c
@@ -1840,7 +1840,7 @@
             len++;
 
         } else {
-            if (len == 0) {
+            if (len == 0 || len > 255) {
                 return NGX_DECLINED;
             }
 
@@ -1851,6 +1851,10 @@
         p--;
     }
 
+    if (len == 0 || len > 255) {
+        return NGX_DECLINED;
+    }
+
     *p = (u_char) len;
 
     return NGX_OK;