net: delete QemuOpts when net_client_init() fails.
commit410cbafebc7168a278a23c856b4f5ff276ef1c85
authorYoshiaki Tamura <tamura.yoshiaki@lab.ntt.co.jp>
Mon, 21 Jun 2010 01:41:36 +0000 (21 10:41 +0900)
committerLuiz Capitulino <lcapitulino@redhat.com>
Thu, 1 Jul 2010 17:27:13 +0000 (1 14:27 -0300)
treea2757a108b6f7490b4cb668d42e2d24607e3c26f
parentbbafc7a8798bc9ed1380e75033544e0614d344c7
net: delete QemuOpts when net_client_init() fails.

This fixes the following scenario using QMP.

First, put a bogus argument "foo" to "type", which results in an error.
{"execute": "netdev_add", "arguments": { "type": "foo", "id": "netdev1" } }
Then, call it again with correct argument "user".
{"execute": "netdev_add", "arguments": { "type": "user", "id": "netdev1" } }
This results in "DuplicatedId" error.

Because the first command was invalid, it should be able to reuse the
same "id", and the second command should work.

Reported-by: Luiz Capitulino <lcapitulino@redhat.com>
Signed-off-by: Yoshiaki Tamura <tamura.yoshiaki@lab.ntt.co.jp>
Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
net.c