[libvirt] [PATCH 9/9] get_nonnull_domain: Drop useless comment

Michal Privoznik mprivozn at redhat.com
Wed Dec 4 09:55:34 UTC 2019


The intent of get_nonnull_domain() is not to validate virDomain
as sent by the client but just to construct the virDomain
structure. The validation is then done in each API when looking
up the domain in our internal hash tables.

Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
---
 src/remote/remote_daemon_dispatch.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/remote/remote_daemon_dispatch.c b/src/remote/remote_daemon_dispatch.c
index f369ffb02a..a96e008242 100644
--- a/src/remote/remote_daemon_dispatch.c
+++ b/src/remote/remote_daemon_dispatch.c
@@ -7240,9 +7240,6 @@ remoteDispatchNetworkPortGetParameters(virNetServerPtr server G_GNUC_UNUSED,
 static virDomainPtr
 get_nonnull_domain(virConnectPtr conn, remote_nonnull_domain domain)
 {
-    /* Should we believe the domain.id sent by the client?  Maybe
-     * this should be a check rather than an assignment? XXX
-     */
     return virGetDomain(conn, domain.name, BAD_CAST domain.uuid, domain.id);
 }
 
-- 
2.23.0




More information about the libvir-list mailing list