Finish effort to decrease maximum stack frame to 2048

[PATCH 5/8] nss: aiforaf: Remove unused 'ret' variable

Posted by Peter Krempa 2 years, 5 months ago

The variable is set but never actually used.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
---
 tools/nss/libvirt_nss.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/tools/nss/libvirt_nss.c b/tools/nss/libvirt_nss.c
index cd2b8feb5a..faa44e78df 100644
--- a/tools/nss/libvirt_nss.c
+++ b/tools/nss/libvirt_nss.c
@@ -469,7 +469,6 @@ aiforaf(const char *name,
         struct addrinfo *pai,
         struct addrinfo **aip)
 {
-    int ret;
     struct hostent resolved;
     char buf[1024] = { 0 };
     int err;
@@ -479,9 +478,9 @@ aiforaf(const char *name,
     struct addrinfo *res;
     char **addrList;

-    if ((ret = NSS_NAME(gethostbyname2)(name, af, &resolved,
-                                        buf, sizeof(buf),
-                                        &err, &herr)) != NS_SUCCESS)
+    if (NSS_NAME(gethostbyname2)(name, af, &resolved,
+                                 buf, sizeof(buf),
+                                 &err, &herr) != NS_SUCCESS)
         return;

     addrList = resolved.h_addr_list;
-- 
2.41.0

Re: [PATCH 5/8] nss: aiforaf: Remove unused 'ret' variable

Posted by Kristina Hanicova 2 years, 5 months ago

On Wed, Aug 30, 2023 at 1:59 PM Peter Krempa <pkrempa@redhat.com> wrote:

> The variable is set but never actually used.
>
> Signed-off-by: Peter Krempa <pkrempa@redhat.com>
> ---
>  tools/nss/libvirt_nss.c | 7 +++----
>  1 file changed, 3 insertions(+), 4 deletions(-)
>

Reviewed-by: Kristina Hanicova <khanicov@redhat.com>

Kristina