Fix safeStrCpy
authorH.G. Muller <h.g.muller@hccnet.nl>
Wed, 17 Nov 2010 12:38:00 +0000 (13:38 +0100)
committerArun Persaud <arun@nubati.net>
Thu, 18 Nov 2010 07:06:48 +0000 (23:06 -0800)
The test for when to terminate a non-fiting string was one off, and the
debug message was written stdoutin stead of debugFP.

backend.c

index 0af00ea..6e544c0 100644 (file)
--- a/backend.c
+++ b/backend.c
@@ -318,11 +318,11 @@ safeStrCpy( char *dst, const char *src, size_t count )
   assert( count > 0 );
 
   for(i=0; i<count; i++) if((dst[i] = src[i]) == NULLCHAR) break;
-  if(  i == count-1 && dst[i] != NULLCHAR)
+  if(  i == count && dst[count-1] != NULLCHAR)
     {
       dst[ count-1 ] = '\0'; // make sure incomplete copy still null-terminated
       if(appData.debugMode)
-      printf("safeStrCpy: copying %s into %s didn't work, not enough space %d\n",src,dst,count);
+      fprintf(debugFP, "safeStrCpy: copying %s into %s didn't work, not enough space %d\n",src,dst,count);
     }
 
   return dst;