[libvirt] [PATCH] libvirt: remove link to outdated mailing list post

Daniel P. Berrangé berrange at redhat.com
Fri Jul 5 13:42:48 UTC 2019


The virConnectGetType() method has an unfortunate signature, returning a
static string that must not be freed by the caller. The remote driver,
however, gets this string dynamically over an RPC call, which raised a
design discussion on the mailing list. Eventually the problem was
resolved by having the remote driver cache the returned string
internally and free it when the connection was closed.

The link to the mailing list is thus talking about a problem that does
not actually exist in the final implementation, and at best serves to
confuse the reader into thinking there might be a memory leak.

Signed-off-by: Daniel P. Berrangé <berrange at redhat.com>
---
 src/libvirt-host.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/libvirt-host.c b/src/libvirt-host.c
index 2978825d22..e5c4e5f72a 100644
--- a/src/libvirt-host.c
+++ b/src/libvirt-host.c
@@ -98,9 +98,6 @@ virConnectSupportsFeature(virConnectPtr conn, int feature)
  * hypervisor, use virConnectGetCapabilities().
  *
  * Returns NULL in case of error, a static zero terminated string otherwise.
- *
- * See also:
- * http://www.redhat.com/archives/libvir-list/2007-February/msg00096.html
  */
 const char *
 virConnectGetType(virConnectPtr conn)
-- 
2.21.0




More information about the libvir-list mailing list