Changeset f85ed4b in mainline for uspace/srv/net/dnsres/dns_msg.c

Timestamp:

2012-08-12T17:51:26Z (13 years ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

08a6382

Parents:

adae30d

Message:

Fixes: initialize off, increment lsize.

File:

• uspace/srv/net/dnsres/dns_msg.c (modified) (5 diffs)

uspace/srv/net/dnsres/dns_msg.c

@@ -45 +45 @@
 #define NAME  "dnsres"
 
+#include <stdio.h>
 static int dns_name_encode(char *name, uint8_t *buf, size_t buf_size,
     size_t *act_size)
@@ -55 +56 @@
         pi = 0;
         di = 1;
-
+        off = 0;
+
+        printf("dns_name_encode(name='%s', buf=%p, buf_size=%zu, act_size=%p\n",
+            name, buf, buf_size, act_size);
         lsize = 0;
         while (true) {
+                printf("off=%zu\n", off);
                 c = str_decode(name, &off, STR_NO_LIMIT);
+                printf("c=%d\n", (int)c);
                 if (c > 127) {
                         /* Non-ASCII character */
+                        printf("non-ascii character\n");
                         return EINVAL;
                 }
@@ -66 +73 @@
                 if (c == '.' || c == '\0') {
                         /* Empty string, starting with period or two consecutive periods. */
-                        if (lsize == 0)
+                        if (lsize == 0) {
+                                printf("empty token\n");
                                 return EINVAL;
+                        }
 
                         if (lsize > DNS_LABEL_MAX_SIZE) {
                                 /* Label too long */
+                                printf("label too long\n");
                                 return EINVAL;
                         }
@@ -83 +93 @@
                         ++di;
                 } else {
+                        ++lsize;
                         if (buf != NULL && di < buf_size)
                                 buf[di++] = c;
@@ -116 +127 @@
 
         *act_size = name_size + sizeof(uint16_t) + sizeof(uint16_t);
+        if (buf == NULL)
+                return EOK;
+
         di = name_size;