diff options
| author | Dimitry Andric <dim@FreeBSD.org> | 2020-01-31 18:13:00 +0000 |
|---|---|---|
| committer | Dimitry Andric <dim@FreeBSD.org> | 2020-01-31 18:13:00 +0000 |
| commit | 31868fab2829d9cb3b1bc972280e5b1c0364a77b (patch) | |
| tree | f66cc589f480a4f078b97d31ccc99256c0882302 /usr.sbin | |
| parent | 43e8403953cb97ffa6306dd13d2e92fdec91e47c (diff) | |
| parent | 303c0aad7d27c15932addb4bbe43f969ae212755 (diff) | |
| download | src-31868fab2829d9cb3b1bc972280e5b1c0364a77b.tar.gz src-31868fab2829d9cb3b1bc972280e5b1c0364a77b.zip | |
Merge r357339 from the clang1000-import branch:
Fix the following -Werror warning from clang 10.0.0 in bsnmpd:
usr.sbin/bsnmpd/modules/snmp_pf/pf_snmp.c:1661:4: error: misleading indentation; statement is not part of the previous 'else' [-Werror,-Wmisleading-indentation]
return (-1);
^
usr.sbin/bsnmpd/modules/snmp_pf/pf_snmp.c:1658:5: note: previous statement is here
} else
^
The intent was to group the return statement with the previous syslog()
call.
MFC after: 3 days
Notes
Notes:
svn path=/head/; revision=357340
Diffstat (limited to 'usr.sbin')
| -rw-r--r-- | usr.sbin/bsnmpd/modules/snmp_pf/pf_snmp.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/usr.sbin/bsnmpd/modules/snmp_pf/pf_snmp.c b/usr.sbin/bsnmpd/modules/snmp_pf/pf_snmp.c index d20f7bccfe26..a14ab5841dfa 100644 --- a/usr.sbin/bsnmpd/modules/snmp_pf/pf_snmp.c +++ b/usr.sbin/bsnmpd/modules/snmp_pf/pf_snmp.c @@ -1655,10 +1655,11 @@ altq_is_enabled(int pfdev) syslog(LOG_INFO, "No ALTQ support in kernel\n" "ALTQ related functions disabled\n"); return (0); - } else + } else { syslog(LOG_ERR, "DIOCGETALTQS returned an error: %s", strerror(errno)); return (-1); + } } return (1); } |