d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Do minimal work necessary to cure a -Wunused-but-set-variable warning from gcc

Browse Source 
How errno is saved before and restored after strtoul calls needs a rethink

MFC after: 1 week
Reported by: gcc 5.x
Sponsored by: EMC / Isilon Storage Division
This commit is contained in:
Enji Cooper 2016-05-14 23:13:23 +00:00
parent baed0627b7
commit 2c0046434b
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=299774
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
usr.sbin/bsnmpd/tools/libbsnmptools/bsnmptc.c
View File

@ -778,11 +778,11 @@ parse_ntp_ts(struct snmp_value *sv, char *val)
saved_errno = errno;
v = strtoul(val, &endptr, 10);
if (errno != 0 || (v / 1000) > 9) {
saved_errno = errno;
errno = saved_errno;
warnx("Integer value %s not supported", val);
return (-1);
} else
saved_errno = errno;
errno = saved_errno;
if (*endptr != '.') {
warnx("Failed reading octet - %s", val);
@ -799,11 +799,11 @@ parse_ntp_ts(struct snmp_value *sv, char *val)
saved_errno = errno;
v = strtoul(val, &endptr, 10);
if (errno != 0 || (v / 1000) > 9) {
saved_errno = errno;
errno = saved_errno;
warnx("Integer value %s not supported", val);
return (-1);
} else
saved_errno = errno;
errno = saved_errno;
for (i = 0, d = 1000; i < 4; i++) {
ntp_ts[i + 4] = v / d;