Commit 8b0d77f1 authored by Cong Wang's avatar Cong Wang Committed by Linus Torvalds

block/compat_ioctl.c: do not leak info to user-space

There is a hole in struct hd_geometry, so we have to zero the struct on
stack before copying it to user-space.
Signed-off-by: default avatarCong Wang <amwang@redhat.com>
Cc: Jens Axboe <axboe@kernel.dk>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 31bd8fbb
...@@ -59,6 +59,7 @@ static int compat_hdio_getgeo(struct gendisk *disk, struct block_device *bdev, ...@@ -59,6 +59,7 @@ static int compat_hdio_getgeo(struct gendisk *disk, struct block_device *bdev,
if (!disk->fops->getgeo) if (!disk->fops->getgeo)
return -ENOTTY; return -ENOTTY;
memset(&geo, 0, sizeof(geo));
/* /*
* We need to set the startsect first, the driver may * We need to set the startsect first, the driver may
* want to override it. * want to override it.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment