[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 0/3] Avoid abort on QMP attempt to add an object with dupl
From: |
Paolo Bonzini |
Subject: |
Re: [PATCH v4 0/3] Avoid abort on QMP attempt to add an object with duplicate id |
Date: |
Mon, 29 Jun 2020 17:30:47 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.6.0 |
On 29/06/20 13:23, Eric Auger wrote:
> Attempting to add an object through QMP with an id that is
> already used leads to a qemu abort. This is a regression since
> d2623129a7de ("qom: Drop parameter @errp of object_property_add()
> & friends").
>
> The first patch fixes the issue and the second patch adds a test
> to check the error is gracefully returned to the QMP client.
>
> The last patch can be considered independently. It merges all the
> object-add tests into a single test function and cover new failure
> cases.
>
> Best Regards
>
> Eric
>
> This series can be found at:
> https://github.com/eauger/qemu/tree/qom-graceful-v4
>
> History:
> - v3 -> v4:
> - addressed style comment from Markus
> - added patch 3
>
> - v2 -> v3:
> - don't take the object reference on failure in
> object_property_try_add_child
> - add g_assert_nonnull(resp) in 2/2 while keeping
> Thomas A-b
>
> - v1 -> v2:
> - use the try terminology.
> - turn object_property_try_add() into a non-static function
> - add the test
>
>
> Eric Auger (3):
> qom: Introduce object_property_try_add_child()
> tests/qmp-cmd-test: Add qmp/object-add-duplicate-id
> tests/qmp-cmd-test: Add qmp/object-add-failure-modes
>
> include/qom/object.h | 26 +++++++++-
> qom/object.c | 21 ++++++--
> qom/object_interfaces.c | 7 ++-
> tests/qtest/qmp-cmd-test.c | 99 ++++++++++++++++++++++++++++++++++++--
> 4 files changed, 140 insertions(+), 13 deletions(-)
>
Very nice. I see a failure reported by Patchew, I'll look into it as
well if you don't have time.
Paolo