diff options
author | Xin LI <delphij@FreeBSD.org> | 2009-09-28 10:22:46 +0000 |
---|---|---|
committer | Xin LI <delphij@FreeBSD.org> | 2009-09-28 10:22:46 +0000 |
commit | 13dcbd75c1d56ac70e5238f9ee62ebeea1cc1725 (patch) | |
tree | 3e2804cc3d7af45960b7b8689f0140710d4d02ac /sys/kern/kern_event.c | |
parent | 66981e7fec77e63c2731ccf84a8c7ac65336cbb9 (diff) | |
download | src-13dcbd75c1d56ac70e5238f9ee62ebeea1cc1725.tar.gz src-13dcbd75c1d56ac70e5238f9ee62ebeea1cc1725.zip |
Use correct sizeof() object for klist 'list'. Currently, struct klist
contained only SLIST_HEAD as its member, thus sizeof(struct klist) would
equal to sizeof(struct klist *), so this change makes the code more
correct in terms of semantics, but should be a no-op to compiler at this
time.
Reported by: MQ <antinvidia at gmail com>
Notes
Notes:
svn path=/head/; revision=197575
Diffstat (limited to 'sys/kern/kern_event.c')
-rw-r--r-- | sys/kern/kern_event.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/sys/kern/kern_event.c b/sys/kern/kern_event.c index 86a950f9bd89..478432e20727 100644 --- a/sys/kern/kern_event.c +++ b/sys/kern/kern_event.c @@ -1229,7 +1229,7 @@ kqueue_expand(struct kqueue *kq, struct filterops *fops, uintptr_t ident, size = kq->kq_knlistsize; while (size <= fd) size += KQEXTENT; - list = malloc(size * sizeof list, M_KQUEUE, mflag); + list = malloc(size * sizeof(*list), M_KQUEUE, mflag); if (list == NULL) return ENOMEM; KQ_LOCK(kq); @@ -1239,13 +1239,13 @@ kqueue_expand(struct kqueue *kq, struct filterops *fops, uintptr_t ident, } else { if (kq->kq_knlist != NULL) { bcopy(kq->kq_knlist, list, - kq->kq_knlistsize * sizeof list); + kq->kq_knlistsize * sizeof(*list)); free(kq->kq_knlist, M_KQUEUE); kq->kq_knlist = NULL; } bzero((caddr_t)list + - kq->kq_knlistsize * sizeof list, - (size - kq->kq_knlistsize) * sizeof list); + kq->kq_knlistsize * sizeof(*list), + (size - kq->kq_knlistsize) * sizeof(*list)); kq->kq_knlistsize = size; kq->kq_knlist = list; } |