[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 2/7] hw/mips/mips_jazz.c: Store irq array in
From: |
Michael Tokarev |
Subject: |
Re: [Qemu-devel] [PATCH v2 2/7] hw/mips/mips_jazz.c: Store irq array in MachineState to fix memory leak |
Date: |
Thu, 04 Jun 2015 18:14:42 +0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.7.0 |
04.06.2015 18:09, Shannon Zhao wrote:
> On 2015/6/4 22:57, Michael Tokarev wrote:
>> 04.06.2015 17:51, Shannon Zhao wrote:
>>> >Yeah, but I think something like below would work.
>>> >
>>> > cpu_exit_irq = &qemu_allocate_irq(cpu_request_exit, NULL, 0);
>>> > DMA_init(0, cpu_exit_irq);
>> NO!!!:)
>
> Hmm...really?
Really-really :)
You're getting an address of a temporary local variable
returned from qemu_allocate_irq, and store that address
in some pointer to be used later. This address is not
valid once we return from this function.
/mjt