[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH] API: Init conn in case of it might be used uninitialized



于 2011年09月02日 13:25, Wen Congyang 写道:
At 09/02/2011 12:52 PM, Osier Yang Write:
There is a goto before "conn" is initialized.
---
  src/libvirt.c |    3 ++-
  1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/src/libvirt.c b/src/libvirt.c
index 4284954..eca919a 100644
--- a/src/libvirt.c
+++ b/src/libvirt.c
@@ -15225,12 +15225,13 @@ virDomainMigrateGetMaxSpeed(virDomainPtr domain,
          return -1;
      }

+    conn = domain->conn;
+
      if (!bandwidth) {
          virLibDomainError(VIR_ERR_INVALID_ARG, __FUNCTION__);
          goto error;
      }

-    conn = domain->conn;
      if (conn->flags&  VIR_CONNECT_RO) {
          virLibDomainError(VIR_ERR_OPERATION_DENIED, __FUNCTION__);
          goto error;
Ack

Thanks, pushed.

Osier


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]