[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] ide: refuse WIN_READ_NATIVE_MAX on empty device
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH] ide: refuse WIN_READ_NATIVE_MAX on empty device |
Date: |
Wed, 10 Apr 2013 11:08:54 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Fri, Apr 05, 2013 at 11:51:01AM +0200, Stefan Hajnoczi wrote:
> What is the highest addressable sector on an empty CD-ROM? Nothing is
> addressable so produce an error.
>
> This patch prevents a divide-by-zero in ide_set_sector() since
> s->sectors and s->heads would be 0. Not to mention that a sector=-1
> argument would be nonsense.
>
> Note that WIN_READ_NATIVE_MAX can be triggered using hdparm -N 1024
> /dev/cdrom. The LBA bit will be set to 1 though, so the only easy way
> to go down the ide_set_sector() CHS code path which divides by zero is
> to comment out the s->select & 0x40 case for testing.
>
> Signed-off-by: Stefan Hajnoczi <address@hidden>
> ---
> hw/ide/core.c | 4 ++++
> 1 file changed, 4 insertions(+)
Applied to my block tree:
https://github.com/stefanha/qemu/commits/block
Stefan
Re: [Qemu-devel] [PATCH] ide: refuse WIN_READ_NATIVE_MAX on empty device,
Stefan Hajnoczi <=