amqp: use the right buffer size for sstrncpy
authorRuben Kerkhof <ruben@rubenkerkhof.com>
Sat, 28 Nov 2015 17:48:20 +0000 (18:48 +0100)
committerRuben Kerkhof <ruben@rubenkerkhof.com>
Sat, 28 Nov 2015 17:48:20 +0000 (18:48 +0100)
buffer is a char *, not an array, so size_of returns the size
of the pointer.

src/amqp.c

index aba4f01..4206bdc 100644 (file)
@@ -199,11 +199,11 @@ static char *camqp_strerror (camqp_config_t *conf, /* {{{ */
     switch (r.reply_type)
     {
         case AMQP_RESPONSE_NORMAL:
-            sstrncpy (buffer, "Success", sizeof (buffer));
+            sstrncpy (buffer, "Success", buffer_size);
             break;
 
         case AMQP_RESPONSE_NONE:
-            sstrncpy (buffer, "Missing RPC reply type", sizeof (buffer));
+            sstrncpy (buffer, "Missing RPC reply type", buffer_size);
             break;
 
         case AMQP_RESPONSE_LIBRARY_EXCEPTION:
@@ -215,7 +215,7 @@ static char *camqp_strerror (camqp_config_t *conf, /* {{{ */
                 return (sstrerror (r.library_error, buffer, buffer_size));
 #endif
             else
-                sstrncpy (buffer, "End of stream", sizeof (buffer));
+                sstrncpy (buffer, "End of stream", buffer_size);
             break;
 
         case AMQP_RESPONSE_SERVER_EXCEPTION: