blkdebug: Fix config with multiple states
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v2 Latest4 patches 1 replyIn review
- v13 patches 3 repliesSuperseded
Changes in v2 from the cover letter
- Use Markus' qemu_opts_reset for freeing QemuOpts. Makes the code reusable and fixes a use-after-free bug.
Patches in v2 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/4 | qemu-option: New qemu_opts_reset()qemu-option.c qemu-option.h | +10 | 0 replies | |
| 2/4 | blkdebug: Fix set_state_opts definitionblock/ | +1 | 0 replies | |
| 3/4 | blkdebug: Free QemuOpts after having read the configblock/ | +2 | 0 replies | |
| 4/4 | blkdebug: Initialize state as 1block/ | +3 | 0 replies |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.
Cover letter [Qemu-devel] [PATCH v2 0/4] blkdebug: Fix config with multiple states
Turns out that using more than one state doesn't really work well. I'm trying to reproduce a bug for which I need states, so now is the time to fix it.
v2:
- Use Markus' qemu_opts_reset for freeing QemuOpts. Makes the code reusable and fixes a use-after-free bug.
Kevin Wolf (3):
blkdebug: Fix set_state_opts definition
blkdebug: Free QemuOpts after having read the config
blkdebug: Initialize state as 1
Markus Armbruster (1):
qemu-option: New qemu_opts_reset()
block/blkdebug.c | 7 ++++++- qemu-option.c | 9 +++++++++ qemu-option.h | 1 + 3 files changed, 16 insertions(+), 1 deletions(-)