* [PATCH v2] nbd: fix uninitialized variable warning
@ 2020-01-08 2:51 pannengyuan
2020-01-08 3:44 ` Richard Henderson
2020-01-08 15:03 ` Laurent Vivier
0 siblings, 2 replies; 3+ messages in thread
From: pannengyuan @ 2020-01-08 2:51 UTC (permalink / raw)
To: eblake
Cc: zhang.zhanghailiang, qemu-block, qemu-trivial, Pan Nengyuan,
qemu-devel, Euler Robot
From: Pan Nengyuan <pannengyuan@huawei.com>
Fixes:
/mnt/sdb/qemu/nbd/server.c: In function 'nbd_handle_request':
/mnt/sdb/qemu/nbd/server.c:2313:9: error: 'ret' may be used uninitialized in this function [-Werror=maybe-uninitialized]
int ret;
Reported-by: Euler Robot <euler.robot@huawei.com>
Signed-off-by: Pan Nengyuan <pannengyuan@huawei.com>
---
Changes v2 to v1:
- change 'if(client->export_meta.bitmap)' into 'else' to fix uninitialized warning and clean up pointless code (suggested by Eric Blake)
---
nbd/server.c | 10 +---------
1 file changed, 1 insertion(+), 9 deletions(-)
diff --git a/nbd/server.c b/nbd/server.c
index 24ebc1a805..87fcd2e7bf 100644
--- a/nbd/server.c
+++ b/nbd/server.c
@@ -2384,20 +2384,12 @@ static coroutine_fn int nbd_handle_request(NBDClient *client,
!client->export_meta.bitmap,
NBD_META_ID_BASE_ALLOCATION,
errp);
- if (ret < 0) {
- return ret;
- }
- }
-
- if (client->export_meta.bitmap) {
+ } else { /* client->export_meta.bitmap */
ret = nbd_co_send_bitmap(client, request->handle,
client->exp->export_bitmap,
request->from, request->len,
dont_fragment,
true, NBD_META_ID_DIRTY_BITMAP, errp);
- if (ret < 0) {
- return ret;
- }
}
return ret;
--
2.21.0.windows.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v2] nbd: fix uninitialized variable warning
2020-01-08 2:51 [PATCH v2] nbd: fix uninitialized variable warning pannengyuan
@ 2020-01-08 3:44 ` Richard Henderson
2020-01-08 15:03 ` Laurent Vivier
1 sibling, 0 replies; 3+ messages in thread
From: Richard Henderson @ 2020-01-08 3:44 UTC (permalink / raw)
To: pannengyuan, eblake
Cc: qemu-trivial, Euler Robot, zhang.zhanghailiang, qemu-block, qemu-devel
On 1/8/20 12:51 PM, pannengyuan@huawei.com wrote:
> From: Pan Nengyuan <pannengyuan@huawei.com>
>
> Fixes:
> /mnt/sdb/qemu/nbd/server.c: In function 'nbd_handle_request':
> /mnt/sdb/qemu/nbd/server.c:2313:9: error: 'ret' may be used uninitialized in this function [-Werror=maybe-uninitialized]
> int ret;
>
> Reported-by: Euler Robot <euler.robot@huawei.com>
> Signed-off-by: Pan Nengyuan <pannengyuan@huawei.com>
> ---
> Changes v2 to v1:
> - change 'if(client->export_meta.bitmap)' into 'else' to fix uninitialized warning and clean up pointless code (suggested by Eric Blake)
> ---
> nbd/server.c | 10 +---------
> 1 file changed, 1 insertion(+), 9 deletions(-)
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
r~
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] nbd: fix uninitialized variable warning
2020-01-08 2:51 [PATCH v2] nbd: fix uninitialized variable warning pannengyuan
2020-01-08 3:44 ` Richard Henderson
@ 2020-01-08 15:03 ` Laurent Vivier
1 sibling, 0 replies; 3+ messages in thread
From: Laurent Vivier @ 2020-01-08 15:03 UTC (permalink / raw)
To: pannengyuan, eblake
Cc: qemu-trivial, Euler Robot, zhang.zhanghailiang, qemu-block, qemu-devel
Le 08/01/2020 à 03:51, pannengyuan@huawei.com a écrit :
> From: Pan Nengyuan <pannengyuan@huawei.com>
>
> Fixes:
> /mnt/sdb/qemu/nbd/server.c: In function 'nbd_handle_request':
> /mnt/sdb/qemu/nbd/server.c:2313:9: error: 'ret' may be used uninitialized in this function [-Werror=maybe-uninitialized]
> int ret;
>
> Reported-by: Euler Robot <euler.robot@huawei.com>
> Signed-off-by: Pan Nengyuan <pannengyuan@huawei.com>
> ---
> Changes v2 to v1:
> - change 'if(client->export_meta.bitmap)' into 'else' to fix uninitialized warning and clean up pointless code (suggested by Eric Blake)
> ---
> nbd/server.c | 10 +---------
> 1 file changed, 1 insertion(+), 9 deletions(-)
>
> diff --git a/nbd/server.c b/nbd/server.c
> index 24ebc1a805..87fcd2e7bf 100644
> --- a/nbd/server.c
> +++ b/nbd/server.c
> @@ -2384,20 +2384,12 @@ static coroutine_fn int nbd_handle_request(NBDClient *client,
> !client->export_meta.bitmap,
> NBD_META_ID_BASE_ALLOCATION,
> errp);
> - if (ret < 0) {
> - return ret;
> - }
> - }
> -
> - if (client->export_meta.bitmap) {
> + } else { /* client->export_meta.bitmap */
> ret = nbd_co_send_bitmap(client, request->handle,
> client->exp->export_bitmap,
> request->from, request->len,
> dont_fragment,
> true, NBD_META_ID_DIRTY_BITMAP, errp);
> - if (ret < 0) {
> - return ret;
> - }
> }
>
> return ret;
>
Reviewed-by: Laurent Vivier <laurent@vivier.eu>
And applied to my trivial-patches branch.
Thanks,
Laurent
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-01-08 15:08 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-08 2:51 [PATCH v2] nbd: fix uninitialized variable warning pannengyuan
2020-01-08 3:44 ` Richard Henderson
2020-01-08 15:03 ` Laurent Vivier
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.