> -----Original Message----- > From: Olaf Hering [mailto:o...@aepfle.de] > Sent: Thursday, August 01, 2013 9:04 AM > To: KY Srinivasan; gre...@linuxfoundation.org > Cc: linux-kernel@vger.kernel.org; Olaf Hering > Subject: [PATCH] Tools: hv: check return value of system in hv_kvp_daemon > > 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 <o...@aepfle.de> Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> > --- > tools/hv/hv_kvp_daemon.c | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/tools/hv/hv_kvp_daemon.c b/tools/hv/hv_kvp_daemon.c > index ba075e5..b96eccc 100644 > --- a/tools/hv/hv_kvp_daemon.c > +++ b/tools/hv/hv_kvp_daemon.c > @@ -1353,7 +1353,11 @@ setval_done: > */ > > snprintf(cmd, sizeof(cmd), "%s %s", "hv_set_ifconfig", if_file); > - system(cmd); > + if (system(cmd)) { > + syslog(LOG_ERR, "Failed to execute cmd '%s'; error: %d %s", > + cmd, errno, strerror(errno)); > + return HV_E_FAIL; > + } > return 0; > > setval_error: > >
-- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
