From 2a3a2c2f5bea5d5cafba3b96c843acd67f791098 Mon Sep 17 00:00:00 2001 From: Michal Privoznik Date: Thu, 5 May 2016 08:42:50 +0200 Subject: [PATCH] virNetServerClientNewPostExecRestart: Drop useless typecasts In this function, @id is defined as unsigned long long. When passing this variable to virJSONValueObjectGetNumberUlong(), well address of this variable, it's typecasted to ull*. There is no need for that. It's a same story with @nrequests_max. Signed-off-by: Michal Privoznik --- src/rpc/virnetserverclient.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/rpc/virnetserverclient.c b/src/rpc/virnetserverclient.c index d38f42140d..d3a3a18f93 100644 --- a/src/rpc/virnetserverclient.c +++ b/src/rpc/virnetserverclient.c @@ -485,7 +485,7 @@ virNetServerClientPtr virNetServerClientNewPostExecRestart(virJSONValuePtr objec return NULL; } if (virJSONValueObjectGetNumberUint(object, "nrequests_max", - (unsigned int *)&nrequests_max) < 0) { + &nrequests_max) < 0) { virReportError(VIR_ERR_INTERNAL_ERROR, "%s", _("Missing nrequests_client_max field in JSON state document")); return NULL; @@ -501,8 +501,7 @@ virNetServerClientPtr virNetServerClientNewPostExecRestart(virJSONValuePtr objec /* no ID found in, a new one must be generated */ id = virNetServerNextClientID((virNetServerPtr) opaque); } else { - if (virJSONValueObjectGetNumberUlong(object, "id", - (unsigned long long *) &id) < 0) { + if (virJSONValueObjectGetNumberUlong(object, "id", &id) < 0) { virReportError(VIR_ERR_INTERNAL_ERROR, "%s", _("Malformed id field in JSON state document")); return NULL; -- GitLab