diff options
author | trasz <trasz@FreeBSD.org> | 2013-10-09 12:30:52 +0000 |
---|---|---|
committer | trasz <trasz@FreeBSD.org> | 2013-10-09 12:30:52 +0000 |
commit | e282c243a6abd26e079c63e71f8570e8199e2dca (patch) | |
tree | 2488d8c55964ed72bbf58640528716ccbcc81416 /usr.sbin/ctladm | |
parent | 089479fb9ecc0c054a9f99cabcf665879324770a (diff) | |
download | FreeBSD-src-e282c243a6abd26e079c63e71f8570e8199e2dca.zip FreeBSD-src-e282c243a6abd26e079c63e71f8570e8199e2dca.tar.gz |
Fix two off-by-ones.
Coverity CID: 1087999, 1011375
Approved by: re (glebius)
Sponsored by: FreeBSD Foundation
Diffstat (limited to 'usr.sbin/ctladm')
-rw-r--r-- | usr.sbin/ctladm/ctladm.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/usr.sbin/ctladm/ctladm.c b/usr.sbin/ctladm/ctladm.c index 5517284..6f03de0 100644 --- a/usr.sbin/ctladm/ctladm.c +++ b/usr.sbin/ctladm/ctladm.c @@ -3442,7 +3442,7 @@ cctl_islist_start_element(void *user_data, const char *name, const char **attr) islist = (struct cctl_islist_data *)user_data; cur_conn = islist->cur_conn; islist->level++; - if ((u_int)islist->level > (sizeof(islist->cur_sb) / + if ((u_int)islist->level >= (sizeof(islist->cur_sb) / sizeof(islist->cur_sb[0]))) errx(1, "%s: too many nesting levels, %zd max", __func__, sizeof(islist->cur_sb) / sizeof(islist->cur_sb[0])); @@ -3848,7 +3848,7 @@ cctl_start_element(void *user_data, const char *name, const char **attr) devlist = (struct cctl_devlist_data *)user_data; cur_lun = devlist->cur_lun; devlist->level++; - if ((u_int)devlist->level > (sizeof(devlist->cur_sb) / + if ((u_int)devlist->level >= (sizeof(devlist->cur_sb) / sizeof(devlist->cur_sb[0]))) errx(1, "%s: too many nesting levels, %zd max", __func__, sizeof(devlist->cur_sb) / sizeof(devlist->cur_sb[0])); |