diff options
| author | Jens Axboe <axboe@fb.com> | 2015-11-10 10:14:38 -0700 |
|---|---|---|
| committer | Jens Axboe <axboe@fb.com> | 2015-11-10 10:14:38 -0700 |
| commit | c1c534609fe8a859f9c8108a5591e6e8a97e34d1 (patch) | |
| tree | f064a2591788be8ac0fcaf4315ed38f8e33bff3a /fs/direct-io.c | |
| parent | 15c4f638f3d41bae52105ca4c0c8760afbcbeaab (diff) | |
| download | op-kernel-dev-c1c534609fe8a859f9c8108a5591e6e8a97e34d1.zip op-kernel-dev-c1c534609fe8a859f9c8108a5591e6e8a97e34d1.tar.gz | |
direct-io: be sure to assign dio->bio_bdev for both paths
btrfs sets ->submit_io(), and we failed to set the block dev for
that path. That resulted in a potential NULL dereference when
we later wait for IO in dio_await_one().
Reported-by: kernel test robot <ying.huang@linux.intel.com>
Signed-off-by: Jens Axboe <axboe@fb.com>
Diffstat (limited to 'fs/direct-io.c')
| -rw-r--r-- | fs/direct-io.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/fs/direct-io.c b/fs/direct-io.c index 7025029..15381c4 100644 --- a/fs/direct-io.c +++ b/fs/direct-io.c @@ -399,14 +399,14 @@ static inline void dio_bio_submit(struct dio *dio, struct dio_submit *sdio) if (dio->is_async && dio->rw == READ && dio->should_dirty) bio_set_pages_dirty(bio); + dio->bio_bdev = bio->bi_bdev; + if (sdio->submit_io) { sdio->submit_io(dio->rw, bio, dio->inode, sdio->logical_offset_in_bio); dio->bio_cookie = BLK_QC_T_NONE; - } else { + } else dio->bio_cookie = submit_bio(dio->rw, bio); - dio->bio_bdev = bio->bi_bdev; - } sdio->bio = NULL; sdio->boundary = 0; |
