Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 13 Jul 2009 14:28:08 -0400
From:      John Baldwin <jhb@freebsd.org>
To:        freebsd-hackers@freebsd.org
Cc:        Nate Eldredge <neldredge@math.ucsd.edu>, Alan Cox <alc@freebsd.org>, Alexander Best <alexbestms@math.uni-muenster.de>
Subject:   Re: mmap/munmap with zero length
Message-ID:  <200907131428.08923.jhb@freebsd.org>
In-Reply-To: <permail-200907050732251e86ffa8000022a8-a_best01@message-id.uni-muenster.de>
References:  <permail-200907050732251e86ffa8000022a8-a_best01@message-id.uni-muenster.de>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sunday 05 July 2009 3:32:25 am Alexander Best wrote:
> so mmap differs from the POSIX recommendation right. the malloc.conf option
> seems more like a workaround/hack. imo it's confusing to have mmap und munmap
> deal differently with len=0. being able to succesfully alocate memory which
> cannot be removed doesn't seem logical to me.

This should fix it:

--- //depot/user/jhb/acpipci/vm/vm_mmap.c
+++ /home/jhb/work/p4/acpipci/vm/vm_mmap.c
@@ -229,7 +229,7 @@

        fp = NULL;
        /* make sure mapping fits into numeric range etc */
-       if ((ssize_t) uap->len < 0 ||
+       if ((ssize_t) uap->len <= 0 ||
            ((flags & MAP_ANON) && uap->fd != -1))
                return (EINVAL);

-- 
John Baldwin



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