doh! properly handle case when src is empty - thanks valgrind!

kthakore · Nov 17, 2008 · fef9979 · fef9979
1 parent 56a9033
commit fef9979
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/server/tools.c b/server/tools.c
@@ -111,12 +111,10 @@ void * memdup(void *src, size_t size)
 size_t strconcat(char *dst, const char *src, size_t size)
 {
         char *ptr = dst + strlen(dst);
-        while (ptr - dst < size - 1) {
+        while (ptr - dst < size - 1 && *src) {
                 *ptr = *src;
                 ptr++;
                 src++;
-                if (!*src)
-                        break;
         }
         *ptr = '\0';
         return ptr - dst;