[src] cvs commit: src/sys/vm vm_contig.c
Nate Lawson
nate at root.org
Fri Mar 5 10:12:46 PST 2004
On Thu, 4 Mar 2004, Alan Cox wrote:
> FreeBSD src repository
>
> Modified files:
> sys/vm vm_contig.c
> Log:
> In the last revision, I introduced a physical contiguity check that is both
> unnecessary and wrong. While it is necessary to verify that the page is
> still free after dropping and reacquiring the free page queue lock, the
> physical contiguity of the page can not change, making this check
> unnecessary. This check was wrong in that it could cause an out-of-bounds
> array access.
>
> Tested by: rwatson
>
> Revision Changes Path
> 1.32 +1 -3 src/sys/vm/vm_contig.c
Reported, tested by: marcel
Initial incorrect fix: njl
-Nate
More information about the cvs-src
mailing list