diff options
| author | delphij <delphij@FreeBSD.org> | 2009-09-28 10:22:46 +0000 |
|---|---|---|
| committer | delphij <delphij@FreeBSD.org> | 2009-09-28 10:22:46 +0000 |
| commit | fd5f08e3e82b6caa8a7fff6a32c9e56b9212003e (patch) | |
| tree | 3e2804cc3d7af45960b7b8689f0140710d4d02ac /sys/kern/kern_event.c | |
| parent | ae17fd849b4e18599a64d781304566dcd2dfa0e4 (diff) | |
| download | FreeBSD-src-fd5f08e3e82b6caa8a7fff6a32c9e56b9212003e.zip FreeBSD-src-fd5f08e3e82b6caa8a7fff6a32c9e56b9212003e.tar.gz | |
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>
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 86a950f..478432e 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; } |
