[PATCH] fix SG_IO bio leak
authorFUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Mon, 11 Dec 2006 09:01:34 +0000 (10:01 +0100)
committerJens Axboe <axboe@nelson.home.kernel.dk>
Tue, 12 Dec 2006 09:22:23 +0000 (10:22 +0100)
This patch fixes bio leaks in SG_IO. rq->bio can be changed after io
completion, so we need to reset rq->bio before calling blk_rq_unmap_user()

http://marc.theaimsgroup.com/?l=linux-kernel&m=116570666807983&w=2

Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
block/scsi_ioctl.c

index b3e210723a71f637605852b7d8136ef594af95d2..045cabd3d4b4b1a6559ade2b9c2da1b8a24d0172 100644 (file)
@@ -228,6 +228,7 @@ static int sg_io(struct file *file, request_queue_t *q,
        struct request *rq;
        char sense[SCSI_SENSE_BUFFERSIZE];
        unsigned char cmd[BLK_MAX_CDB];
+       struct bio *bio;
 
        if (hdr->interface_id != 'S')
                return -EINVAL;
@@ -308,6 +309,7 @@ static int sg_io(struct file *file, request_queue_t *q,
        if (ret)
                goto out;
 
+       bio = rq->bio;
        rq->retries = 0;
 
        start_time = jiffies;
@@ -338,6 +340,7 @@ static int sg_io(struct file *file, request_queue_t *q,
                        hdr->sb_len_wr = len;
        }
 
+       rq->bio = bio;
        if (blk_rq_unmap_user(rq))
                ret = -EFAULT;