From: H.G. Muller <h.g.muller@hccnet.nl>
Date: Wed, 17 Nov 2010 12:38:00 +0000 (+0100)
Subject: Fix safeStrCpy
X-Git-Url: http://winboard.nl/cgi-bin?a=commitdiff_plain;h=cf7256e228782b533112473ff6977a72528e793a;hp=d41436d65b74f8bf75e7de167eb1f3ea701c893c;p=xboard.git

Fix safeStrCpy

The test for when to terminate a non-fiting string was one off, and the
debug message was written stdoutin stead of debugFP.
---

diff --git a/backend.c b/backend.c
index 0af00ea..6e544c0 100644
--- 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;