<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Aug 30, 2023 at 2:21 PM Peter Krempa <<a href="mailto:pkrempa@redhat.com">pkrempa@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Wed, Aug 30, 2023 at 13:59:21 +0200, Peter Krempa wrote:<br>
> The 'buf', 'sa' and 'hints' stack allocated helper variables are never<br>
> used together. Decrease the stack memory usage by scoping them off into<br>
> do-while blocks.<br>
> <br>
> In this instance we do not want to use dynamic allocation as this is the<br>
> NSS module.<br>
> <br>
> Signed-off-by: Peter Krempa <<a href="mailto:pkrempa@redhat.com" target="_blank">pkrempa@redhat.com</a>><br>
> ---<br>
>  tools/nss/libvirt_nss.c | 97 +++++++++++++++++++++++------------------<br>
>  1 file changed, 54 insertions(+), 43 deletions(-)<br>
> <br>
> diff --git a/tools/nss/libvirt_nss.c b/tools/nss/libvirt_nss.c<br>
> index 37720bf4ae..dff3c034bf 100644<br>
> --- a/tools/nss/libvirt_nss.c<br>
> +++ b/tools/nss/libvirt_nss.c<br>
<br>
[...]<br>
<br>
> -        hints = *pai;<br>
> -        hints.ai_flags = AI_NUMERICHOST;<br>
> -        hints.ai_family = af;<br>
> +            hints = *pai;<br>
> +            hints.ai_flags = AI_NUMERICHOST;<br>
> +            hints.ai_family = af;<br>
> <br>
> -        if (getaddrinfo(host, NULL, &hints, &res0)) {<br>
> -            addrList++;<br>
> -            continue;<br>
<br>
Ehh, self-NACK ...<span class="gmail_default" style="font-size:small"></span></blockquote><div><br></div><div style="font-size:small" class="gmail_default">Apart from moving these two continue into loops...<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
<br>
> -        }<br>
> +            if (getaddrinfo(host, NULL, &hints, &res0)) {<br>
> +                addrList++;<br>
> +                continue;<br>
> +            }<br>
> +        } while (false);<br>
> <br>
>          for (res = res0; res; res = res->ai_next)<br>
>              res->ai_flags = pai->ai_flags;<br>
> -- <br>
> 2.41.0<br>
> <br>
<br></blockquote><div><br></div><div style="font-size:small" class="gmail_default"><div>Reviewed-by: Kristina Hanicova <<a href="mailto:khanicov@redhat.com" target="_blank">khanicov@redhat.com</a>> <br></div><font color="#888888" style="--darkreader-inline-color: #b2a696;"><font color="#888888" style="--darkreader-inline-color: #b2a696;"><div><br></div><div><div class="gmail_default"><font size="2">Kristina</font></div></div></font></font></div></div></div>