On 02/01/2016 05:31 AM, Markus Armbruster wrote:
>> |+visit_start_struct(v, NULL, NULL, "ACPI_DEVICE_OST", 0, );
>> | if (err) {
>> | goto out;
>> | }
>> | visit_type_ACPIOSTInfo(v, , "info", );
>> | if (err) {
>> |-goto out;
>> |+goto out_obj;
>> |
Eric Blake writes:
> All other successful clients of visit_start_struct() were paired
> with an unconditional visit_end_struct(); but the generated
> code for events was relying on qmp_output_visitor_cleanup() to
> work on an incomplete visit. Alter the code to guarantee that
Eric Blake writes:
> On 02/01/2016 05:31 AM, Markus Armbruster wrote:
>
>>> |+visit_start_struct(v, NULL, NULL, "ACPI_DEVICE_OST", 0, );
>>> | if (err) {
>>> | goto out;
>>> | }
>>> | visit_type_ACPIOSTInfo(v, , "info", );
>>> | if (err) {
>>> |-
All other successful clients of visit_start_struct() were paired
with an unconditional visit_end_struct(); but the generated
code for events was relying on qmp_output_visitor_cleanup() to
work on an incomplete visit. Alter the code to guarantee that
the struct is completed, which will make a