[PATCH v4 0/4] Fix a few minor issues reported by Coverity
Sergei Heifetz <[email protected]>
| Newsgroups | org.nongnu.qemu-trivial,org.nongnu.qemu-devel |
|---|---|
| Message-ID | <[email protected]> |
Hi. This patch series fixes several minor issues reported by our local Coverity installation. Changes in v4: - Included Peter's explanation why `assert` is redundant in the commit message of the third patch. - Added `Reviewed-by` tags that I missed in v2 to the other three patches. Changes in v3: - Nothing. I sent the second version of the series without the corresponding 'v2' suffix by mistake. I apologise. Changes in v2: - Remove assert(block) in system/physmem.c instead of placing it before the dereference. Sergei Heifetz (4): migration/savevm.c: reorder usage and assertion of mis->from_src_file dump/dump.c: reorder usage and assertion of block system/physmem.c: remove useless assertion of block hw/usb/core.c: reorder usage and assertion of p->ep dump/dump.c | 2 +- hw/usb/core.c | 2 +- migration/savevm.c | 3 ++- system/physmem.c | 2 -- 4 files changed, 4 insertions(+), 5 deletions(-) -- 2.34.1