Code

Replace mmap with xmmap, better handling MAP_FAILED.
authorShawn O. Pearce <spearce@spearce.org>
Sun, 24 Dec 2006 05:47:23 +0000 (00:47 -0500)
committerJunio C Hamano <junkio@cox.net>
Fri, 29 Dec 2006 19:36:45 +0000 (11:36 -0800)
commitc4712e4553f13d87d655325a57538f299402d457
treed09717f9d9a811d36ae5572b131a5b40e1dfcdcf
parent97bfeb34df1aa8a1cf232278624a5a5c924ee380
Replace mmap with xmmap, better handling MAP_FAILED.

In some cases we did not even bother to check the return value of
mmap() and just assume it worked.  This is bad, because if we are
out of virtual address space the kernel returned MAP_FAILED and we
would attempt to dereference that address, segfaulting without any
real error output to the user.

We are replacing all calls to mmap() with xmmap() and moving all
MAP_FAILED checking into that single location.  If a mmap call
fails we try to release enough least-recently-used pack windows
to possibly succeed, then retry the mmap() attempt.  If we cannot
mmap even after releasing pack memory then we die() as none of our
callers have any reasonable recovery strategy for a failed mmap.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
config.c
diff.c
git-compat-util.h
read-cache.c
refs.c
sha1_file.c