[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v3 1/9] block: Avoid dead assignment
From: |
Philippe Mathieu-Daudé |
Subject: |
[PATCH v3 1/9] block: Avoid dead assignment |
Date: |
Wed, 22 Apr 2020 15:31:44 +0200 |
Fix warning reported by Clang static code analyzer:
block.c:3167:5: warning: Value stored to 'ret' is never read
ret = bdrv_fill_options(&options, filename, &flags, &local_err);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Fixes: 462f5bcf6
Reported-by: Clang Static Analyzer
Suggested-by: Markus Armbruster <address@hidden>
Reviewed-by: Alistair Francis <address@hidden>
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
---
block.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/block.c b/block.c
index 2e3905c99e..27e7e0e3ab 100644
--- a/block.c
+++ b/block.c
@@ -3171,7 +3171,7 @@ static BlockDriverState *bdrv_open_inherit(const char
*filename,
}
ret = bdrv_fill_options(&options, filename, &flags, &local_err);
- if (local_err) {
+ if (ret < 0) {
goto fail;
}
--
2.21.1
- [PATCH v3 0/9] misc: Trivial static code analyzer fixes, Philippe Mathieu-Daudé, 2020/04/22
- [PATCH v3 2/9] blockdev: Remove dead assignment, Philippe Mathieu-Daudé, 2020/04/22
- [PATCH v3 1/9] block: Avoid dead assignment,
Philippe Mathieu-Daudé <=
- [PATCH v3 4/9] hw/input/adb-kbd: Remove dead assignment, Philippe Mathieu-Daudé, 2020/04/22
- [PATCH v3 3/9] hw/i2c/pm_smbus: Remove dead assignment, Philippe Mathieu-Daudé, 2020/04/22
- [PATCH v3 5/9] hw/ide/sii3112: Remove dead assignment, Philippe Mathieu-Daudé, 2020/04/22
- [PATCH v3 6/9] hw/isa/i82378: Remove dead assignment, Philippe Mathieu-Daudé, 2020/04/22
- [PATCH v3 7/9] hw/gpio/aspeed_gpio: Remove dead assignment, Philippe Mathieu-Daudé, 2020/04/22
- [PATCH v3 9/9] hw/timer/pxa2xx_timer: Add assertion to silent static analyzer warning, Philippe Mathieu-Daudé, 2020/04/22
- [PATCH v3 8/9] hw/timer/stm32f2xx_timer: Remove dead assignment, Philippe Mathieu-Daudé, 2020/04/22