From: Roy Marples <roy@marples.name>
Date: Wed, 14 Nov 2012 10:43:55 +0000 (+0000)
Subject: When decoding RCC3397 domains we should ensure that we have
X-Git-Tag: v5.6.3~5
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=11fe19507bf71d2ab870e51e4d0e8d111ae1aa40;p=thirdparty%2Fdhcpcd.git

When decoding RCC3397 domains we should ensure that we have
decoded something before changing the last '.' to ' ' and
NULL terminating the string.
---

diff --git a/dhcp.c b/dhcp.c
index 35efae30..da620705 100644
--- a/dhcp.c
+++ b/dhcp.c
@@ -439,10 +439,12 @@ get_option_uint8(uint8_t *i, const struct dhcp_message *dhcp, uint8_t option)
 ssize_t
 decode_rfc3397(char *out, ssize_t len, int pl, const uint8_t *p)
 {
+	const char *start;
 	const uint8_t *r, *q = p;
 	int count = 0, l, hops;
 	uint8_t ltype;
 
+	start = out;
 	while (q - p < pl) {
 		r = NULL;
 		hops = 0;
@@ -482,15 +484,19 @@ decode_rfc3397(char *out, ssize_t len, int pl, const uint8_t *p)
 			}
 		}
 		/* change last dot to space */
-		if (out)
+		if (out && out != start)
 			*(out - 1) = ' ';
 		if (r)
 			q = r;
 	}
 
 	/* change last space to zero terminator */
-	if (out)
-		*(out - 1) = 0;
+	if (out) {
+		if (out != start)
+			*(out - 1) = '\0';
+		else if (len > 0)
+			*out = '\0';
+	}
 
 	return count;  
 }