diff options
author | Conrad Meyer <cem@FreeBSD.org> | 2016-04-27 00:10:32 +0000 |
---|---|---|
committer | Conrad Meyer <cem@FreeBSD.org> | 2016-04-27 00:10:32 +0000 |
commit | da95a2ae56aca88dfb389c801e765c3c22eee49a (patch) | |
tree | 2f618d17883e36f645b15409dc332712d1b993d1 /sys/kern | |
parent | a286650b083c91e1aade720986c476369baea2eb (diff) | |
download | src-da95a2ae56aca88dfb389c801e765c3c22eee49a.tar.gz src-da95a2ae56aca88dfb389c801e765c3c22eee49a.zip |
posix4_mib: Don't overrun facility_initialized array
The facility_initialized and facility arrays are the same size and were
intended to be indexed the same. I believe this mismatch was just a
typo/braino in r208731.
Reported by: Coverity
CID: 1017430
Sponsored by: EMC / Isilon Storage Division
Notes
Notes:
svn path=/head/; revision=298678
Diffstat (limited to 'sys/kern')
-rw-r--r-- | sys/kern/posix4_mib.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sys/kern/posix4_mib.c b/sys/kern/posix4_mib.c index a2be3ff9ebfb..d2a8582424fb 100644 --- a/sys/kern/posix4_mib.c +++ b/sys/kern/posix4_mib.c @@ -113,9 +113,9 @@ p31b_sysctl_proc(SYSCTL_HANDLER_ARGS) num = arg2; if (!P31B_VALID(num)) return (EINVAL); - val = facility_initialized[num] ? facility[num - 1] : 0; + val = facility_initialized[num - 1] ? facility[num - 1] : 0; error = sysctl_handle_int(oidp, &val, 0, req); - if (error == 0 && req->newptr != NULL && facility_initialized[num]) + if (error == 0 && req->newptr != NULL && facility_initialized[num - 1]) facility[num - 1] = val; return (error); } @@ -137,7 +137,7 @@ p31b_unsetcfg(int num) { facility[num - 1] = 0; - facility_initialized[num -1] = 0; + facility_initialized[num - 1] = 0; } int |