[ros-diffs] [arty] 36419: Fix possible overrun of the last byte of dbuf (coverity 552).

Author: arty
Date: Tue Sep 23 03:43:49 2008
New Revision: 36419
URL: http://svn.reactos.org/svn/reactos?rev=36419&view=rev
Log:
Fix possible overrun of the last byte of dbuf (coverity 552).
Modified:
    trunk/reactos/base/services/dhcp/dhclient.c
Modified: trunk/reactos/base/services/dhcp/dhclient.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/base/services/dhcp/dhclien…
==============================================================================
--- trunk/reactos/base/services/dhcp/dhclient.c [iso-8859-1] (original)
+++ trunk/reactos/base/services/dhcp/dhclient.c [iso-8859-1] Tue Sep 23 03:43:49 2008
@@ -480,9 +480,9 @@
             RegSetValueEx( RegKey, "DhcpNameServer", 0, REG_SZ,
                            (LPBYTE)nsbuf, strlen(nsbuf) + 1);
-
-            free( nsbuf );
         }
+       // free(NULL) is defined to be OK too
+       free( nsbuf );
     }
 }
@@ -1706,7 +1706,7 @@
                                        len = ip->client->
                                            config->defaults[i].len +
                                            lease->options[i].len;
-                                       if (len > sizeof(dbuf)) {
+                                       if (len >= sizeof(dbuf)) {
                                                warning("no space to %s %s",
                                                    "prepend option",
                                                    dhcp_options[i].name);