Fix waitpid() call to only run in error case (merge error)

ChangeLog

+Wed Sep 19 17:42:00 EST 2007 Daniel P. Berrange <berrange@redhat.com>
+
+	* src/remote_internal.c: Fix waitpid() call to only be done
+	in doRemoteOpen error case (merge error from previous commit)
+
 Wed Sep 19 13:39:00 EST 2007 Daniel P. Berrange <berrange@redhat.com>
 
 	* configure.in: Use pkg-config to locate gnutls if pkg-config

src/remote_internal.c

@@ -647,14 +647,14 @@ doRemoteOpen (virConnectPtr conn, struct private_data *priv, const char *uri_str
         if (priv->uses_tls && priv->session)
             gnutls_bye (priv->session, GNUTLS_SHUT_RDWR);
         close (priv->sock);
-    }
-    if (priv->pid > 0) {
-        pid_t reap;
-        do {
-            reap = waitpid(priv->pid, NULL, 0);
-            if (reap == -1 && errno == EINTR)
-                continue;
-        } while (reap != -1 && reap != priv->pid);
+        if (priv->pid > 0) {
+            pid_t reap;
+            do {
+                reap = waitpid(priv->pid, NULL, 0);
+                if (reap == -1 && errno == EINTR)
+                    continue;
+            } while (reap != -1 && reap != priv->pid);
+        }
     }
 
     /* Free up the URL and strings. */