diff mbox

block: Release allocated options after bdrv_open

Message ID 4BD9B2D2.9040208@siemens.com
State New
Headers show

Commit Message

Jan Kiszka April 29, 2010, 4:24 p.m. UTC 
They aren't used afterwards nor supposed to be stored by a bdrv_create
handler.

Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
---
 block.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

Comments

Kevin Wolf April 30, 2010, 7:58 a.m. UTC | #1 
Am 29.04.2010 18:24, schrieb Jan Kiszka:
> They aren't used afterwards nor supposed to be stored by a bdrv_create
> handler.
> 
> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>

Thanks, applied to the block branch.

Kevin
diff mbox

Patch

diff --git a/block.c b/block.c
index 7974215..f228d8d 100644
--- a/block.c
+++ b/block.c
@@ -411,6 +411,7 @@  int bdrv_open(BlockDriverState *bs, const char *filename, int flags,
         }
 
         ret = bdrv_create(bdrv_qcow2, tmp_filename, options);
+        free_option_parameters(options);
         if (ret < 0) {
             return ret;
         }