diff mbox series

[7/8] sunrpc/netname.c: fix warn unused result

Message ID 20230418121130.844302-8-fberat@redhat.com
State Superseded
Headers
Series Fix warn unused result |

Checks

Context Check Description
dj/TryBot-apply_patch success Patch applied to master at the time it was sent

Commit Message

Frederic Berat April 18, 2023, 12:11 p.m. UTC 
  Fix unused result warnings, detected when _FORTIFY_SOURCE is enabled in
glibc.
---
 sunrpc/netname.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Siddhesh Poyarekar April 18, 2023, 12:51 p.m. UTC | #1 
On 2023-04-18 08:11, Frédéric Bérat via Libc-alpha wrote:
> Fix unused result warnings, detected when _FORTIFY_SOURCE is enabled in
> glibc.
> ---
>   sunrpc/netname.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/sunrpc/netname.c b/sunrpc/netname.c
> index dd2cb1b441..949954959a 100644
> --- a/sunrpc/netname.c
> +++ b/sunrpc/netname.c
> @@ -96,7 +96,8 @@ host2netname (char netname[MAXNETNAMELEN + 1], const char *host,
>         else
>   	{
>   	  domainname[0] = 0;
> -	  getdomainname (domainname, MAXHOSTNAMELEN);
> +	  if(getdomainname (domainname, MAXHOSTNAMELEN))

Missing space after if.

> +		  return 0;
>   	}
>       }
>     else
diff mbox series

Patch

diff --git a/sunrpc/netname.c b/sunrpc/netname.c
index dd2cb1b441..949954959a 100644
--- a/sunrpc/netname.c
+++ b/sunrpc/netname.c
@@ -96,7 +96,8 @@  host2netname (char netname[MAXNETNAMELEN + 1], const char *host,
       else
 	{
 	  domainname[0] = 0;
-	  getdomainname (domainname, MAXHOSTNAMELEN);
+	  if(getdomainname (domainname, MAXHOSTNAMELEN))
+		  return 0;
 	}
     }
   else