blktrace: use BLKTRACE_BDEV_SIZE as the name size for setup structure
authorJens Axboe <jens.axboe@oracle.com>
Wed, 1 Oct 2008 14:16:25 +0000 (16:16 +0200)
committerJens Axboe <jens.axboe@oracle.com>
Thu, 9 Oct 2008 06:56:20 +0000 (08:56 +0200)
Define as 32, which is is what BDEVNAME_SIZE is/was as well. This keeps
the user interface the same and gets rid of the difference between
kernel and user api here.

Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
block/blktrace.c
include/linux/blktrace_api.h

index 9e0212c90b29eb0c3d1fd29e0aed9affdc4aa1e4..85049a7e7a179a97c283eb4ebe6c1fe7285f80cf 100644 (file)
@@ -369,7 +369,8 @@ int do_blk_trace_setup(struct request_queue *q, char *name, dev_t dev,
        if (!buts->buf_size || !buts->buf_nr)
                return -EINVAL;
 
-       strcpy(buts->name, name);
+       strncpy(buts->name, name, BLKTRACE_BDEV_SIZE);
+       buts->name[BLKTRACE_BDEV_SIZE - 1] = '\0';
 
        /*
         * some device names have larger paths - convert the slashes
index a2a7d0ca27587d007baef289589566e374fb72ec..3a31eb506164a814a2b9599d11d709f9915ed1cc 100644 (file)
@@ -122,15 +122,13 @@ enum {
        Blktrace_stopped,
 };
 
+#define BLKTRACE_BDEV_SIZE     32
+
 /*
  * User setup structure passed with BLKTRACESTART
  */
 struct blk_user_trace_setup {
-#ifdef __KERNEL__
-       char name[BDEVNAME_SIZE];       /* output */
-#else
-       char name[32];                  /* output */
-#endif
+       char name[BLKTRACE_BDEV_SIZE];  /* output */
        __u16 act_mask;                 /* input */
        __u32 buf_size;                 /* input */
        __u32 buf_nr;                   /* input */