2013-08-01 12:43:17

by Olaf Hering

Subject: [PATCH] Tools: hv: check return value of daemon to fix compiler warning.

hv_kvp_daemon.c: In function 'main':
hv_kvp_daemon.c:1441:8: warning: ignoring return value of 'daemon', declared with attribute warn_unused_result [-Wunused-result]

Signed-off-by: Olaf Hering <[email protected]>
---
tools/hv/hv_kvp_daemon.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tools/hv/hv_kvp_daemon.c b/tools/hv/hv_kvp_daemon.c
index 657c1d2..418ac55 100644
--- a/tools/hv/hv_kvp_daemon.c
+++ b/tools/hv/hv_kvp_daemon.c
@@ -1439,7 +1439,8 @@ int main(void)
char *kvp_recv_buffer;
size_t kvp_recv_buffer_len;

- daemon(1, 0);
+ if (daemon(1, 0))
+ return 1;
openlog("KVP", 0, LOG_USER);
syslog(LOG_INFO, "KVP starting; pid is:%d", getpid());

2013-08-01 14:35:20

by KY Srinivasan

Subject: RE: [PATCH] Tools: hv: check return value of daemon to fix compiler warning.

> -----Original Message-----
> From: Olaf Hering [mailto:[email protected]]
> Sent: Thursday, August 01, 2013 8:43 AM
> To: KY Srinivasan; [email protected]
> Cc: [email protected]; Olaf Hering
> Subject: [PATCH] Tools: hv: check return value of daemon to fix compiler
> warning.
>
> hv_kvp_daemon.c: In function 'main':
> hv_kvp_daemon.c:1441:8: warning: ignoring return value of 'daemon', declared
> with attribute warn_unused_result [-Wunused-result]
>
> Signed-off-by: Olaf Hering <[email protected]>
Signed-off-by: K. Y. Srinivasan <[email protected]>

> ---
> tools/hv/hv_kvp_daemon.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/tools/hv/hv_kvp_daemon.c b/tools/hv/hv_kvp_daemon.c
> index 657c1d2..418ac55 100644
> --- a/tools/hv/hv_kvp_daemon.c
> +++ b/tools/hv/hv_kvp_daemon.c
> @@ -1439,7 +1439,8 @@ int main(void)
> char *kvp_recv_buffer;
> size_t kvp_recv_buffer_len;
>
> - daemon(1, 0);
> + if (daemon(1, 0))
> + return 1;
> openlog("KVP", 0, LOG_USER);
> syslog(LOG_INFO, "KVP starting; pid is:%d", getpid());
>
>
>