mirror of https://gitlab.com/qemu-project/qemu
You cannot select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
626b39006d
When SET_STREAM_FORMAT is called, the st->buft timer is overwritten, thus
causing a memory leak. This was originally fixed in commit 816139ae6a5
("hw/audio/hda: fix memory leak on audio setup", 2024-11-14) but that
caused the audio to break in SPICE.
Fortunately, a simpler fix is possible. The timer only needs to be
reset, because the callback is always the same (st->output is set at
realize time in hda_audio_init); call to timer_new_ns overkill. Replace
it with timer_del and only initialize the timer once; for simplicity,
do it even if use_timer is false.
An even simpler fix would be to free the old time in hda_audio_setup().
However, it seems better to place the initialization of the timer close
to that of st->ouput.
Cc: qemu-stable@nongnu.org
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Michael Tokarev <mjt@tls.msk.ru>
Message-ID: <20241114125318.1707590-3-pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
|
2 weeks ago | |
|---|---|---|
| .. | ||
| Kconfig | 1 year ago | |
| ac97.c | 3 months ago | |
| ac97.h | 2 years ago | |
| adlib.c | 1 year ago | |
| asc.c | 7 months ago | |
| cs4231.c | 3 months ago | |
| cs4231a.c | 3 months ago | |
| es1370.c | 3 months ago | |
| fmopl.c | 1 year ago | |
| fmopl.h | 1 year ago | |
| gus.c | 11 months ago | |
| gusemu.h | 8 years ago | |
| gusemu_hal.c | 1 year ago | |
| gusemu_mixer.c | 5 years ago | |
| gustate.h | 9 years ago | |
| hda-codec-common.h | 11 years ago | |
| hda-codec.c | 2 weeks ago | |
| intel-hda-defs.h | 1 year ago | |
| intel-hda.c | 3 months ago | |
| intel-hda.h | 1 year ago | |
| lm4549.c | 11 months ago | |
| lm4549.h | 1 year ago | |
| marvell_88w8618.c | 3 months ago | |
| meson.build | 1 year ago | |
| pcspk.c | 11 months ago | |
| pl041.c | 3 months ago | |
| pl041.h | 9 years ago | |
| pl041.hx | ||
| sb16.c | 11 months ago | |
| soundhw.c | 1 month ago | |
| trace-events | 3 months ago | |
| trace.h | 4 years ago | |
| via-ac97.c | 3 months ago | |
| virtio-snd-pci.c | 1 year ago | |
| virtio-snd.c | 2 months ago | |
| wm8750.c | 11 months ago | |