[Qemu-devel] [PULL for-3.0 0/2] qemu-ga patch queue for hard-freeze

[Qemu-devel] [PULL for-3.0 0/2] qemu-ga patch queue for hard-freeze

[Michael Roth]

The following changes since commit 102ad0a80f5110483efd06877c29c4236be267f9:

  Merge remote-tracking branch 'remotes/armbru/tags/pull-misc-2018-07-16' into staging (2018-07-16 15:34:38 +0100)

are available in the git repository at:

  git://github.com/mdroth/qemu.git tags/qga-pull-2018-07-16-tag

for you to fetch changes up to d9c745c1768727aadd77e352c85114bc61e67bd4:

  qga: fix file descriptor leak (2018-07-16 15:05:58 -0500)

----------------------------------------------------------------
qemu-ga patch queue for hard-freeze

* fix file descriptor leak in suspend command
* fix leak in guest-get-fsinfo

----------------------------------------------------------------
Marc-André Lureau (1):
      qga: fix 'driver' leak in guest-get-fsinfo

Paolo Bonzini (1):
      qga: fix file descriptor leak

 qga/commands-posix.c | 2 ++
 1 file changed, 2 insertions(+)

[Qemu-devel] [PULL for-3.0 1/2] qga: fix 'driver' leak in guest-get-fsinfo

[Michael Roth]

From: Marc-André Lureau <marcandre.lureau@redhat.com>

'driver' is leaked when the loop is not broken.

Leak introduced by commit 743c71d03c20d64f2bae5fba6f26cdf5e4b1bda6,
spotted by ASAN.

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
---
 qga/commands-posix.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/qga/commands-posix.c b/qga/commands-posix.c
index 233f78a406..c46767b0dd 100644
--- a/qga/commands-posix.c
+++ b/qga/commands-posix.c
@@ -890,6 +890,7 @@ static void build_guest_fsinfo_for_real_device(char const *syspath,
             break;
         }
 
+        g_free(driver);
         if (sscanf(p, "/%x:%x:%x.%x%n",
                           pci, pci + 1, pci + 2, pci + 3, &pcilen) == 4) {
             p += pcilen;
-- 
2.11.0

[Qemu-devel] [PULL for-3.0 2/2] qga: fix file descriptor leak

[Michael Roth]

From: Paolo Bonzini <pbonzini@redhat.com>

The file descriptor for /sys/power/state was never closed.  Reported
by Coverity.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
---
 qga/commands-posix.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/qga/commands-posix.c b/qga/commands-posix.c
index c46767b0dd..37e8a2d791 100644
--- a/qga/commands-posix.c
+++ b/qga/commands-posix.c
@@ -1652,6 +1652,7 @@ static bool linux_sys_state_supports_mode(SuspendMode mode, Error **errp)
     }
 
     ret = read(fd, buf, sizeof(buf) - 1);
+    close(fd);
     if (ret <= 0) {
         return false;
     }
-- 
2.11.0

Re: [Qemu-devel] [PULL for-3.0 0/2] qemu-ga patch queue for hard-freeze

[Peter Maydell]

On 16 July 2018 at 22:44, Michael Roth <mdroth@linux.vnet.ibm.com> wrote:
> The following changes since commit 102ad0a80f5110483efd06877c29c4236be267f9:
>
>   Merge remote-tracking branch 'remotes/armbru/tags/pull-misc-2018-07-16' into staging (2018-07-16 15:34:38 +0100)
>
> are available in the git repository at:
>
>   git://github.com/mdroth/qemu.git tags/qga-pull-2018-07-16-tag
>
> for you to fetch changes up to d9c745c1768727aadd77e352c85114bc61e67bd4:
>
>   qga: fix file descriptor leak (2018-07-16 15:05:58 -0500)
>
> ----------------------------------------------------------------
> qemu-ga patch queue for hard-freeze
>
> * fix file descriptor leak in suspend command
> * fix leak in guest-get-fsinfo
>
> ----------------------------------------------------------------
> Marc-André Lureau (1):
>       qga: fix 'driver' leak in guest-get-fsinfo
>
> Paolo Bonzini (1):
>       qga: fix file descriptor leak
>
>  qga/commands-posix.c | 2 ++
>  1 file changed, 2 insertions(+)

Applied, thanks.

-- PMM

[Qemu-devel] [PULL for-3.0 0/2] qemu-ga patch queue for hard-freeze

[Michael Roth]

The following changes since commit 4eecc2e212bae7308d8582d99f665dfc253c9dda:

  Merge remote-tracking branch 'remotes/rth/tags/pull-tcg-20180723' into staging (2018-07-23 19:24:05 +0100)

are available in the Git repository at:

  git://github.com/mdroth/qemu.git tags/qga-pull-2018-07-23-tag

for you to fetch changes up to ae7da1e5f658ea21d96e565514de20ff2cf24fa1:

  qga: process_event() simplification and leak fix (2018-07-23 18:54:26 -0500)

----------------------------------------------------------------
qemu-ga patch queue for hard-freeze

* fix leak in qga main loop error path
* better error reporting when Windows version doesn't support fstrim

----------------------------------------------------------------
Marc-André Lureau (1):
      qga: process_event() simplification and leak fix

Sameeh Jubran (1):
      qga-win: Handle fstrim for OSes lower than Win8

 qga/commands-win32.c | 13 +++++++++++++
 qga/main.c           | 54 ++++++++++++++++++++++++++--------------------------
 2 files changed, 40 insertions(+), 27 deletions(-)

Re: [Qemu-devel] [PULL for-3.0 0/2] qemu-ga patch queue for hard-freeze

[Peter Maydell]

On 24 July 2018 at 01:01, Michael Roth <mdroth@linux.vnet.ibm.com> wrote:
> The following changes since commit 4eecc2e212bae7308d8582d99f665dfc253c9dda:
>
>   Merge remote-tracking branch 'remotes/rth/tags/pull-tcg-20180723' into staging (2018-07-23 19:24:05 +0100)
>
> are available in the Git repository at:
>
>   git://github.com/mdroth/qemu.git tags/qga-pull-2018-07-23-tag
>
> for you to fetch changes up to ae7da1e5f658ea21d96e565514de20ff2cf24fa1:
>
>   qga: process_event() simplification and leak fix (2018-07-23 18:54:26 -0500)
>
> ----------------------------------------------------------------
> qemu-ga patch queue for hard-freeze
>
> * fix leak in qga main loop error path
> * better error reporting when Windows version doesn't support fstrim
>
> ----------------------------------------------------------------
> Marc-André Lureau (1):
>       qga: process_event() simplification and leak fix
>
> Sameeh Jubran (1):
>       qga-win: Handle fstrim for OSes lower than Win8
>
>  qga/commands-win32.c | 13 +++++++++++++
>  qga/main.c           | 54 ++++++++++++++++++++++++++--------------------------
>  2 files changed, 40 insertions(+), 27 deletions(-)


Applied, thanks.

-- PMM