mirror of
https://github.com/xen-project/xen.git
synced 2026-10-29 18:50:05 +01:00
tools/xenstore: fix use after free bug in xenstore_control
There is a very unlikely use after free bug and a memory leak in
live_update_start() of xenstore_control. Fix those.
Coverity-Id: 1472399
Fixes: 7f97193e6a ("tools/xenstore: add live update command to xenstore-control")
Signed-off-by: Juergen Gross <jgross@suse.com>
Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>
This commit is contained in:
1 parent
452ddbe359
commit
1d24e551b9
1 file changed
+5
@@ -43,7 +43,12 @@ static int live_update_start(struct xs_handle *xsh, bool force, unsigned int to)
|
||||
if (len < 0)
|
||||
return 1;
|
||||
|
||||
ret = strdup("BUSY");
|
||||
if (!ret)
|
||||
return 1;
|
||||
|
||||
for (time_start = time(NULL); time(NULL) - time_start < to;) {
|
||||
free(ret);
|
||||
ret = xs_control_command(xsh, "live-update", buf, len);
|
||||
if (!ret)
|
||||
goto err;
|
||||
|
||||