vfs_fruit: mmap under FreeBSD needs PROT_READ
authorVolker Lendecke <vl@samba.org>
Thu, 8 Jan 2015 16:14:37 +0000 (17:14 +0100)
committerJeremy Allison <jra@samba.org>
Thu, 8 Jan 2015 22:38:06 +0000 (23:38 +0100)
We memmove, which does read

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Ralph Böhme <rb@sernet.de>
Reviewed-by: Jeremy Allison <jra@samba.org>
source3/modules/vfs_fruit.c

index e2d9b35dbee2fe2d1de462fdc448ee9502b2e66b..4eace1eb35491f45e84df582457f2c85c66703e2 100644 (file)
@@ -652,7 +652,7 @@ static int ad_convert(struct adouble *ad, int fd)
                ad_getentrylen(ad, ADEID_RFORK);
 
        /* FIXME: direct use of mmap(), vfs_aio_fork does it too */
-       map = mmap(NULL, origlen, PROT_WRITE, MAP_SHARED, fd, 0);
+       map = mmap(NULL, origlen, PROT_READ|PROT_WRITE, MAP_SHARED, fd, 0);
        if (map == MAP_FAILED) {
                DEBUG(2, ("mmap AppleDouble: %s\n", strerror(errno)));
                rc = -1;