Fix remote_id parsing in option82_plugin (contributed by [email protected])

HISTORY

@@ -1,6 +1,11 @@
+2013-06-19:
+	4.7 version
+	* fix remote_id parsing in option82_plugin (contributed by [email protected])
+
 2013-04-15:
 	4.6 version
 	* fix rps_limit converting it to per-interface basis (and removing mutex)
+
 2012-12-03:
 	4.5 version
 	* destroy() in plugins for smooth shutdown.

dhcprelya.c

@@ -50,7 +50,7 @@
 
 #include "dhcprelya.h"
 
-#define VERSION "4.6"
+#define VERSION "4.7"
 
 /* options */
 /* globals (can check in modules) */

option82_plugin.c

@@ -92,12 +92,13 @@ option82_plugin_init(plugin_options_head_t *options_head)
 			/* is a string */
 			if (*p == '"') {
 				p++;
-				for (i = 0; *p != '"' && *p != '\0'; p++, i++)
-					rid[i] = *p;
+				for(rid_len=0; *p != '"' && *p != '\0' && rid_len<sizeof(rid); p++,rid_len++)
+					rid[rid_len] = *p;
 				if (*p != '"') {
 					logd(LOG_ERR, "option82_plugin: Syntex error in option value at line: %s", opts->option_line);
 					return 0;
 				}
+				rid[rid_len] = '\0';
 			} else if (strcasecmp(p, "0x") == 0) {
 				p += 2;
 				logd(LOG_ERR, "option82_plugin: hexadecimal is not supported yet at line: %s", opts->option_line);