Statistics
| Branch: | Revision:

root / qga @ 834574ea

Latest revisions

# Date Author Comment
24a53049 01/30/2013 12:14 pm Markus Armbruster

qemu-ga: Fix unchecked strdup() by converting to g_strdup()

I figure it's freed somewhere deep down in QAPI, with g_free().

Signed-off-by: Markus Armbruster <>
Reviewed-by: Eric Blake <>
Reviewed-by: Luiz Capitulino <>...

10a2158f 01/28/2013 09:46 pm Markus Armbruster

qemu-ga: Plug leaks on qmp_guest_network_get_interfaces() error paths

Spotted by Coverity.

Signed-off-by: Markus Armbruster <>
Reviewed-by: Eric Blake <>
Reviewed-by: Luiz Capitulino <>
Signed-off-by: Michael Roth <>

6f686749 01/28/2013 09:46 pm Markus Armbruster

qemu-ga: Plug memory leak in guest_fsfreeze_cleanup()

Neglects to free errors allocated by qmp_guest_fsfreeze_thaw().
Spotted by Coverity.

While there, drop the test whether return value is negative (it's
never true), and improve logging.

Signed-off-by: Markus Armbruster <>...

e4ada482 01/19/2013 12:24 pm Stefan Weil

Replace non-portable asprintf by g_strdup_printf

g_strdup_printf already handles OOM errors, so some error handling in
QEMU code can be removed.

Signed-off-by: Stefan Weil <>
Signed-off-by: Blue Swirl <>

2fd3402d 01/16/2013 12:50 am Anthony Liguori

Merge remote-tracking branch 'mdroth/qga-pull-1-14-2013-2' into staging

  • mdroth/qga-pull-1-14-2013-2:
    qga: add missing commas in json docs

Signed-off-by: Anthony Liguori <>

01b87f6d 01/15/2013 12:15 am Eric Blake

qga: add missing commas in json docs

  • qga/qapi-schema.json: Use valid JSON.

Signed-off-by: Eric Blake <>
Signed-off-by: Michael Roth <>

7868181f 01/14/2013 08:08 pm Markus Armbruster

qemu-ga: Handle errors uniformely in ga_channel_open()

We detect errors in several places. One reports with g_error(), which
calls abort(), the others report with g_critical(). Three of them
exit(), three return false.

Always report with g_critical(), and return false....

d4f4a3ef 01/14/2013 08:07 pm Markus Armbruster

qemu-ga: Plug fd leak on ga_channel_open() error paths

Spotted by Coverity.

Signed-off-by: Markus Armbruster <>
Reviewed-by: Eric Blake <>
Reviewed-by: Michael Roth <>
Reviewed-by: Luiz Capitulino <>...

32c16620 01/14/2013 08:07 pm Markus Armbruster

qemu-ga: Plug fd leak on ga_channel_listen_accept() error path

Spotted by Coverity.

Signed-off-by: Markus Armbruster <>
Reviewed-by: Eric Blake <>
Reviewed-by: Michael Roth <>
Reviewed-by: Luiz Capitulino <>...

03ac10f1 01/14/2013 08:07 pm Markus Armbruster

qemu-ga: Plug file descriptor leak on ga_open_pidfile() error path

Spotted by Coverity. Also document why we keep it open on success.

Signed-off-by: Markus Armbruster <>
Reviewed-by: Eric Blake <>
Reviewed-by: Michael Roth <>...

View revisions

Also available in: Atom