[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-stable] [Qemu-devel] [PATCH v3 6/7] qdev: Protect device-list-
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-stable] [Qemu-devel] [PATCH v3 6/7] qdev: Protect device-list-properties against broken devices |
Date: |
Tue, 29 Sep 2015 10:05:27 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Peter Maydell <address@hidden> writes:
> On 28 September 2015 at 20:36, Markus Armbruster <address@hidden> wrote:
>> 1. I made device-introspection-test run "info qom-tree", which has a
>> lovely propensity to crash when a crappy device left dangling pointer
>> behind. This led me to "cgthree", "cuda", "integrator_debug",
>> "macio-oldworld", "macio-newworld", "pxa2xx-fir", "SUNW,tcx". They
>> all create memory regions without owner in their instance_init()
>> method.
>
> I guess these are all just "oops, we forgot to pass the Object* in
> instead of NULL" bugs rather than more difficult fixes.
I'm leaving the actual fixing to people with a better understanding of
these devices, and ability to actually test them.
- Re: [Qemu-stable] [Qemu-devel] [PATCH v3 6/7] qdev: Protect device-list-properties against broken devices,
Markus Armbruster <=