diff options
| author | Philippe Mathieu-Daudé | 2020-10-28 16:08:10 +0100 |
|---|---|---|
| committer | Philippe Mathieu-Daudé | 2020-11-09 00:53:07 +0100 |
| commit | 60f6de8fbafa5dc57af63792e3ba1a910239d195 (patch) | |
| tree | fb26fc9e6115586760dddacd6ab95384a242b626 /hw/rx | |
| parent | MAINTAINERS: Demote Renesas target & hardware to 'Odd Fixes' (diff) | |
| download | qemu-60f6de8fbafa5dc57af63792e3ba1a910239d195.tar.gz qemu-60f6de8fbafa5dc57af63792e3ba1a910239d195.tar.xz qemu-60f6de8fbafa5dc57af63792e3ba1a910239d195.zip | |
hw/rx/rx-gdbsim: Fix memory leak (CID 1432307)
As load_device_tree() returns allocated memory,
we need to free it.
Cc: Yoshinori Sato <ysato@users.sourceforge.jp>
Fixes: bda19d7bb56 ("hw/rx: Add RX GDB simulator")
Reported-by: Coverity (CID 1432307: RESOURCE_LEAK)
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20201102104542.2064745-1-f4bug@amsat.org>
Diffstat (limited to 'hw/rx')
| -rw-r--r-- | hw/rx/rx-gdbsim.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/hw/rx/rx-gdbsim.c b/hw/rx/rx-gdbsim.c index 417ec0564b..285549c79b 100644 --- a/hw/rx/rx-gdbsim.c +++ b/hw/rx/rx-gdbsim.c @@ -122,9 +122,8 @@ static void rx_gdbsim_init(MachineState *machine) if (dtb_filename) { ram_addr_t dtb_offset; int dtb_size; - void *dtb; + g_autofree void *dtb = load_device_tree(dtb_filename, &dtb_size); - dtb = load_device_tree(dtb_filename, &dtb_size); if (dtb == NULL) { error_report("Couldn't open dtb file %s", dtb_filename); exit(1); |