diff options
author | Andrew Morton <akpm@linux-foundation.org> | 2011-05-27 01:25:15 +0200 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2011-05-27 02:12:34 +0200 |
commit | 4ed960b14d3b5fd14f1d9eb02f6d7e398317627a (patch) | |
tree | bcbe96919f4b19e2fd98366385aa41fba84699c4 /Documentation/accounting | |
parent | Documentation/accounting/getdelays.c: fix unused var warning (diff) | |
download | linux-4ed960b14d3b5fd14f1d9eb02f6d7e398317627a.tar.xz linux-4ed960b14d3b5fd14f1d9eb02f6d7e398317627a.zip |
Documentation/accounting/getdelays.c: handle sendto() failures
Fixes
Documentation/accounting/getdelays.c: In function `get_family_id':
Documentation/accounting/getdelays.c:172:14: warning: variable `rc' set but not used [-Wunused-but-set-variable]
Reported-by: "Justin P. Mattock" <justinmattock@gmail.com>
Cc: Balbir Singh <balbir@in.ibm.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'Documentation/accounting')
-rw-r--r-- | Documentation/accounting/getdelays.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/Documentation/accounting/getdelays.c b/Documentation/accounting/getdelays.c index 16e16d57b9eb..fed225401dd2 100644 --- a/Documentation/accounting/getdelays.c +++ b/Documentation/accounting/getdelays.c @@ -177,6 +177,8 @@ static int get_family_id(int sd) rc = send_cmd(sd, GENL_ID_CTRL, getpid(), CTRL_CMD_GETFAMILY, CTRL_ATTR_FAMILY_NAME, (void *)name, strlen(TASKSTATS_GENL_NAME)+1); + if (rc < 0) + return 0; /* sendto() failure? */ rep_len = recv(sd, &ans, sizeof(ans), 0); if (ans.n.nlmsg_type == NLMSG_ERROR || |