Use length guarded sprintf functions
authorJens Axboe <axboe@kernel.dk>
Sat, 1 Oct 2011 04:20:37 +0000 (22:20 -0600)
committerJens Axboe <axboe@kernel.dk>
Sat, 1 Oct 2011 04:20:37 +0000 (22:20 -0600)
Signed-off-by: Jens Axboe <axboe@kernel.dk>
infolog.c
server.c

index 8396522..b7b4a5e 100644 (file)
--- a/infolog.c
+++ b/infolog.c
@@ -12,7 +12,7 @@ int log_info(const char *format, ...)
        size_t len;
 
        va_start(args, format);
-       len = vsprintf(buffer, format, args);
+       len = vsnprintf(buffer, sizeof(buffer), format, args);
        va_end(args);
 
        if (is_backend)
@@ -28,7 +28,7 @@ int log_err(const char *format, ...)
        size_t len;
 
        va_start(args, format);
-       len = vsprintf(buffer, format, args);
+       len = vsnprintf(buffer, sizeof(buffer), format, args);
        va_end(args);
 
        if (is_backend)
index 4a78f38..a70dc14 100644 (file)
--- a/server.c
+++ b/server.c
@@ -143,7 +143,6 @@ struct fio_net_cmd *fio_net_cmd_read(int sk)
                return NULL;
        }
 
-
        return ret;
 }
 
@@ -392,7 +391,7 @@ int fio_server_log(const char *format, ...)
        va_list args;
 
        va_start(args, format);
-       sprintf(buffer, format, args);
+       snprintf(buffer, sizeof(buffer), format, args);
        va_end(args);
 
        return fio_server_text_output(buffer, strlen(buffer));