[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 09/26] iscsi: migrate iscsi driver QemuOptionPar
From: |
Leandro Dorileo |
Subject: |
Re: [Qemu-devel] [PATCH 09/26] iscsi: migrate iscsi driver QemuOptionParameter usage |
Date: |
Fri, 21 Mar 2014 13:54:50 +0000 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Fri, Mar 21, 2014 at 02:42:35PM +0100, Peter Lieven wrote:
> On 21.03.2014 14:31, Leandro Dorileo wrote:
> >On Fri, Mar 21, 2014 at 07:43:44AM +0100, Peter Lieven wrote:
> >>On 21.03.2014 01:13, Leandro Dorileo wrote:
> >>>Do the directly migration from QemuOptionParameter to QemuOpts on
> >>>iscsi block driver.
> >>>
> >>>Signed-off-by: Leandro Dorileo <address@hidden>
> >>>---
> >>> block/iscsi.c | 32 ++++++++++++++++----------------
> >>> 1 file changed, 16 insertions(+), 16 deletions(-)
> >>>
> >>>diff --git a/block/iscsi.c b/block/iscsi.c
> >>>index b490e98..85252e7 100644
> >>>--- a/block/iscsi.c
> >>>+++ b/block/iscsi.c
> >>>@@ -1125,7 +1125,7 @@ static int iscsi_open(BlockDriverState *bs, QDict
> >>>*options, int flags,
> >>> QemuOpts *opts;
> >>> Error *local_err = NULL;
> >>> const char *filename;
> >>>- int i, ret;
> >>>+ int i, ret = 0;
> >>why? is there a chance that ret remains uninitialized?
> >Yep, my compiler tells me so:
> >
> >block/iscsi.c:1128:12: error: ‘ret’ may be used uninitialized in this
> >function [-Werror=maybe-uninitialized]
> >
> >
> >>> if ((BDRV_SECTOR_SIZE % 512) != 0) {
> >>> error_setg(errp, "iSCSI: Invalid BDRV_SECTOR_SIZE. "
> >>>@@ -1382,8 +1382,7 @@ static int iscsi_truncate(BlockDriverState *bs,
> >>>int64_t offset)
> >>> return 0;
> >>> }
> >>>-static int iscsi_create(const char *filename, QEMUOptionParameter
> >>>*options,
> >>>- Error **errp)
> >>>+static int iscsi_create(const char *filename, QemuOpts *options, Error
> >>>**errp)
> >>> {
> >>> int ret = 0;
> >>> int64_t total_size = 0;
> >>>@@ -1393,12 +1392,9 @@ static int iscsi_create(const char *filename,
> >>>QEMUOptionParameter *options,
> >>> bs = bdrv_new("");
> >>>- /* Read out options */
> >>>- while (options && options->name) {
> >>>- if (!strcmp(options->name, "size")) {
> >>>- total_size = options->value.n / BDRV_SECTOR_SIZE;
> >>>- }
> >>>- options++;
> >>>+ total_size = qemu_opt_get_size(options, BLOCK_OPT_SIZE, 0);
> >>>+ if (total_size) {
> >>>+ total_size = total_size / BDRV_SECTOR_SIZE;
> >>> }
> >>you don't need the if condition. 0 / BDRV_SECTOR_SIZE = 0.
> >>
> >I'm not sure, bdrv_img_create() will set BLOCK_OPT_SIZE with img_size, we
> >have no guarantee on the
> >value passed to bdrv_img_create(), we don't check img_size value there,
> >having said that can't
> >we run on division by zero here? The previous code wasn't checking it but I
> >wonder if the problem
> >wasn't there already.
>
> division by zero is x / 0 not 0 / x.
>
> 0 / x = 0
> x / 0 = undef
>
Yep, true.
--
Leandro Dorileo
- [Qemu-devel] [PATCH 04/26] QemuOpt: introduce qemu_opts_append(), (continued)
- [Qemu-devel] [PATCH 04/26] QemuOpt: introduce qemu_opts_append(), Leandro Dorileo, 2014/03/20
- [Qemu-devel] [PATCH 05/26] QemuOpt: add qemu_opt_print_help(), Leandro Dorileo, 2014/03/20
- [Qemu-devel] [PATCH 06/26] block: migrate block later QemuOptionParameter, Leandro Dorileo, 2014/03/20
- [Qemu-devel] [PATCH 07/26] cow: migrate cow driver QemuOptionParameter usage, Leandro Dorileo, 2014/03/20
- [Qemu-devel] [PATCH 08/26] gluster: migrate gluster driver QemuOptionParameter usage, Leandro Dorileo, 2014/03/20
- [Qemu-devel] [PATCH 09/26] iscsi: migrate iscsi driver QemuOptionParameter usage, Leandro Dorileo, 2014/03/20
[Qemu-devel] [PATCH 10/26] nfs: migrate nfs driver QemuOptionParameter usage, Leandro Dorileo, 2014/03/20
[Qemu-devel] [PATCH 11/26] qcow: migrate qcow driver QemuOptionParameter usage, Leandro Dorileo, 2014/03/20
[Qemu-devel] [PATCH 12/26] qcow2: migrate qcow2 driver QemuOptionParameter usage, Leandro Dorileo, 2014/03/20
[Qemu-devel] [PATCH 13/26] qed: migrate qed driver QemuOptionParameter usage, Leandro Dorileo, 2014/03/20
[Qemu-devel] [PATCH 14/26] raw-posix: migrate raw-posix driver QemuOptionParameter usage, Leandro Dorileo, 2014/03/20
[Qemu-devel] [PATCH 15/26] raw-win32: migrate cow driver QemuOptionParameter usage, Leandro Dorileo, 2014/03/20
[Qemu-devel] [PATCH 16/26] raw_bsd: migrate raw_bsd driver QemuOptionParameter usage, Leandro Dorileo, 2014/03/20
[Qemu-devel] [PATCH 21/26] vhdx: migrate vhdx driver QemuOptionParameter usage, Leandro Dorileo, 2014/03/20
[Qemu-devel] [PATCH 22/26] vmdk: migrate vmdk driver QemuOptionParameter usage, Leandro Dorileo, 2014/03/20
[Qemu-devel] [PATCH 23/26] vpc: migrate vpc driver QemuOptionParameter usage, Leandro Dorileo, 2014/03/20