Debug code sometimes copied too much data

17 years ago · 2b715b89f1
parent ddd85ec7e0
commit 2b715b89f1
1 changed files with 4 additions and 5 deletions
--- a/opentracker.c
+++ b/opentracker.c
@ -226,11 +226,10 @@ static void httpresponse( const int64 s, char *data, size_t l ) {
  size_t      reply_size = 0, reply_off;

 #ifdef _DEBUG_HTTPERROR
-  memcpy( debug_request, data, sizeof( debug_request ) );
-  if( l < sizeof( debug_request ) )
+  if( l >= sizeof( debug_request ) )
+    l = sizeof( debug_request) - 1;
+  memcpy( debug_request, data, l );
  debug_request[ l ] = 0;
-  else
-    debug_request[ sizeof( debug_request ) - 1 ] = 0;
 #endif

  /* This one implicitely tests strlen < 5, too -- remember, it is \n terminated */