aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEnji Cooper <ngie@FreeBSD.org>2017-03-13 17:15:45 +0000
committerEnji Cooper <ngie@FreeBSD.org>2017-03-13 17:15:45 +0000
commit513bdaa1418a580a4ad07e6e220086d1daba796c (patch)
tree69d955bc9fe19f5c0995a842f374efaa8c37d8dc
parent6c5b1b394d352bb98fc77d5a290347add9f16cb7 (diff)
downloadsrc-513bdaa1418a580a4ad07e6e220086d1daba796c.tar.gz
src-513bdaa1418a580a4ad07e6e220086d1daba796c.zip
Fix -Wunused-but-set-warning with `ret`
While here, resolve Coverity warnings by demonstrating that vfprintf's return value is being explicitly ignored. MFC after: 1 week Reported by: gcc 6.3.0 Tested with: clang, gcc 4.2.1, gcc 6.3.0 Sponsored by: Dell EMC Isilon
Notes
Notes: svn path=/head/; revision=315200
-rw-r--r--sbin/dhclient/tests/fake.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/sbin/dhclient/tests/fake.c b/sbin/dhclient/tests/fake.c
index c204d49402c5..91bbc6e42792 100644
--- a/sbin/dhclient/tests/fake.c
+++ b/sbin/dhclient/tests/fake.c
@@ -14,7 +14,7 @@ error(char *fmt, ...)
va_list ap;
va_start(ap, fmt);
- vfprintf(stderr, fmt, ap);
+ (void)vfprintf(stderr, fmt, ap);
va_end(ap);
fprintf(stderr, "\n");
@@ -24,11 +24,10 @@ error(char *fmt, ...)
int
warning(char *fmt, ...)
{
- int ret;
va_list ap;
va_start(ap, fmt);
- ret = vfprintf(stderr, fmt, ap);
+ (void)vfprintf(stderr, fmt, ap);
va_end(ap);
fprintf(stderr, "\n");