Author: hiren
Date: Fri Sep 20 15:57:50 2013
New Revision: 255730
URL: http://svnweb.freebsd.org/changeset/base/255730
Log:
Fix a range check and a display string.
Reviewed by: loos
Approved by: sbruno (mentor, implicit)
Approved by: re (glebius)
Modified:
head/sbin/etherswitchcfg/etherswitchcfg.c
Modified: head/sbin/etherswitchcfg/etherswitchcfg.c
==============================================================================
--- head/sbin/etherswitchcfg/etherswitchcfg.c Fri Sep 20 07:45:37 2013
(r255729)
+++ head/sbin/etherswitchcfg/etherswitchcfg.c Fri Sep 20 15:57:50 2013
(r255730)
@@ -274,7 +274,7 @@ set_vlangroup_vid(struct cfg *cfg, char
etherswitch_vlangroup_t vg;
v = strtol(argv[1], NULL, 0);
- if (v < 0 || v >= IEEE802DOT1Q_VID_MAX)
+ if (v < 0 || v > IEEE802DOT1Q_VID_MAX)
errx(EX_USAGE, "vlan must be between 0 and %d",
IEEE802DOT1Q_VID_MAX);
vg.es_vlangroup = cfg->unit;
if (ioctl(cfg->fd, IOETHERSWITCHGETVLANGROUP, &vg) != 0)
@@ -623,7 +623,7 @@ main(int argc, char *argv[])
print_info(&cfg);
} else if (sscanf(argv[0], "port%d", &cfg.unit) == 1) {
if (cfg.unit < 0 || cfg.unit >=
cfg.info.es_nports)
- errx(EX_USAGE, "port unit must be
between 0 and %d", cfg.info.es_nports);
+ errx(EX_USAGE, "port unit must be
between 0 and %d", cfg.info.es_nports - 1);
newmode(&cfg, MODE_PORT);
} else if (sscanf(argv[0], "vlangroup%d", &cfg.unit) ==
1) {
if (cfg.unit < 0 || cfg.unit >=
cfg.info.es_nvlangroups)
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"
