Code

write_redis: pass a literal string to snprintf()
authorMarc Fournier <marc.fournier@camptocamp.com>
Mon, 24 Nov 2014 14:00:45 +0000 (15:00 +0100)
committerMarc Fournier <marc.fournier@camptocamp.com>
Mon, 24 Nov 2014 20:45:06 +0000 (21:45 +0100)
This corrects the following compiler warning when building with
"-Werror=format-security" (the default when building debian packages):

write_redis.c:91:3: error: format not a string literal and no format
arguments

src/write_redis.c

index b47650d178149bdc2d3342d669afaf9c2cf94d4c..b4c5e212e300e83a5d5de5d060f02c0940d9b70c 100644 (file)
@@ -88,8 +88,7 @@ static int wr_write (const data_set_t *ds, /* {{{ */
   }                                                                  \
 } while (0)
 
-  APPEND (time);
-  APPEND (":");
+  APPEND ("%s:", time);
 
   for (i = 0; i < ds->ds_num; i++)
   {