Fix a range check and a display string.

Reviewed by:	loos
Approved by:	sbruno (mentor, implicit)
Approved by:	re (glebius)
This commit is contained in:
Hiren Panchasara 2013-09-20 15:57:50 +00:00
parent 428b7ca290
commit ba5ff393cd
Notes: svn2git 2020-12-20 02:59:44 +00:00
svn path=/head/; revision=255730

View file

@ -274,7 +274,7 @@ set_vlangroup_vid(struct cfg *cfg, char *argv[])
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)