Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 1 Aug 2009 00:17:19 +0200
From:      Ed Schouten <ed@80386.nl>
To:        Dag-Erling =?iso-8859-1?Q?Sm=F8rgrav?= <des@des.no>
Cc:        jmg@FreeBSD.org, d@delphij.net, MQ <antinvidia@gmail.com>, freebsd-arch@FreeBSD.org
Subject:   Re: [PATCH] type issue in kern_event.c
Message-ID:  <20090731221719.GS1292@hoeg.nl>
In-Reply-To: <86ocr062w9.fsf@ds4.des.no>
References:  <4A72B1DC.3040907@delphij.net> <86ocr062w9.fsf@ds4.des.no>

next in thread | previous in thread | raw e-mail | index | archive | help

[-- Attachment #1 --]
Hi Dag-Erling,

* Dag-Erling Smørgrav <des@des.no> wrote:
> Xin LI <delphij@delphij.net> writes:
> > I think these should be fixed, and here is my proposed patch.
> 
> 404 Patch Not Found

Xin Li sent it to me, but I think he has forgotten to add the lists back
to Cc again. Here's the patch he sent to me earlier today.

Index: kern_event.c
===================================================================
--- kern_event.c	(revision 195945)
+++ kern_event.c	(working copy)
@@ -1106,7 +1106,7 @@ kqueue_expand(struct kqueue *kq, struct filterops
 			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);
@@ -1116,13 +1116,13 @@ kqueue_expand(struct kqueue *kq, struct filterops
 			} 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);
+				    (size - kq->kq_knlistsize) * sizeof *list);
 				kq->kq_knlistsize = size;
 				kq->kq_knlist = list;
 			}

-- 
 Ed Schouten <ed@80386.nl>
 WWW: http://80386.nl/

[-- Attachment #2 --]
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (FreeBSD)

iEYEARECAAYFAkpzbW8ACgkQ52SDGA2eCwXWdACeMw2Xu46RripyiwLKklHHEuS0
VQAAnA+CFWNyG2TZTQsU2fbcj1acL+62
=EaP6
-----END PGP SIGNATURE-----

Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20090731221719.GS1292>