Date: Wed, 2 Jun 2004 00:04:21 -0700 (PDT) From: Poul-Henning Kamp <phk@FreeBSD.org> To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/alpha/alpha pmap.c src/sys/i386/i386 pmap.c Message-ID: <200406020704.i5274MMO059017@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
phk 2004/06/02 00:04:21 PDT
FreeBSD src repository
Modified files: (Branch: RELENG_4)
sys/alpha/alpha pmap.c
sys/i386/i386 pmap.c
Log:
There is a comma missing in the table initializing the
pmap_prefault_pageorder array. This has two effects:
1. The resulting bogus contents of the array thwarts part of the
optimization effect pmap_prefault() is supposed to have.
2. The resulting array is only 7 elements long (auto-sized), while
pmap_prefault() expects it to be the intended 8 elements. So this
function in fact accesses memory beyond the end of the array.
Fortunately though, if the data at this location is out of bounds it
will be ignored.
This bug dates back more than 6 years. It has been introduced in
revision 1.178.
Submitted by: Uwe Doering <gemini@geminix.org>
PR: 67460
Revision Changes Path
1.35.2.11 +1 -1 src/sys/alpha/alpha/pmap.c
1.250.2.26 +1 -1 src/sys/i386/i386/pmap.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200406020704.i5274MMO059017>