[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH] qemu-img: avoid overflow of min_sparse paramete
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-block] [PATCH] qemu-img: avoid overflow of min_sparse parameter |
Date: |
Fri, 13 Jul 2018 09:17:43 +0200 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
Am 12.07.2018 um 21:08 hat Peter Lieven geschrieben:
> the min_sparse convert parameter can overflow (e.g. -S 1024G)
> in the conversion from int64_t to int resulting in a negative
> min_sparse parameter. Avoid this by limiting the valid parameters
> to sane values. In fact anything exceeding the convert buffer size
> is also pointless. While at it also forbid values that are non
> multiple of 512 to avoid undesired behaviour. Values between 1 and
> 511 were legal, but resulted in full allocation.
>
> Cc: address@hidden
> Signed-off-by: Peter Lieven <address@hidden>
> ---
> qemu-img.c | 16 +++++++++++-----
> 1 file changed, 11 insertions(+), 5 deletions(-)
>
> diff --git a/qemu-img.c b/qemu-img.c
> index 4a7ce43..2896746 100644
> --- a/qemu-img.c
> +++ b/qemu-img.c
> @@ -2005,6 +2005,8 @@ static int convert_do_copy(ImgConvertState *s)
> return s->ret;
> }
>
> +#define MAX_BUF_SECTORS 32768
> +
> static int img_convert(int argc, char **argv)
> {
> int c, bs_i, flags, src_flags = 0;
> @@ -2100,8 +2102,12 @@ static int img_convert(int argc, char **argv)
> int64_t sval;
>
> sval = cvtnum(optarg);
> - if (sval < 0) {
> - error_report("Invalid minimum zero buffer size for sparse
> output specified");
> + if (sval < 0 || sval & BDRV_SECTOR_BITS ||
BDRV_SECTOR_BITS is 9 (because 1 << 9 == BDRV_SECTOR_SIZE), not a bit
mask to be used with &. I think what you want is BDRV_SECTOR_SIZE - 1.
Kevin