Sweep through the tree fixing mmap() usage:

  - Use MAP_FAILED instead of the constant -1 to indicate
    failure (required by POSIX).
  - Removed flag arguments of '0' (required by POSIX).
  - Fixed code which expected an error return of 0.
  - Fixed code which thought any address with the high bit set
    was an error.
  - Check for failure where no checks were present.

Discussed with:	bde

1 files changed, 2 insertions, 2 deletions

diff --git a/share/examples/meteor/rgb16.c b/share/examples/meteor/rgb16.c
index 17b1592..4c7ee07 100644
--- a/share/examples/meteor/rgb16.c
+++ b/share/examples/meteor/rgb16.c
@@ -77,9 +77,9 @@ main()
 		exit(1);
 	}
 
-        rgb16 = (short *)mmap((caddr_t)0,SIZE,PROT_READ,0, i, (off_t)0);
+        rgb16 = (short *)mmap((caddr_t)0,SIZE,PROT_READ,MAP_SHARED, i, (off_t)0);
 
-	if (rgb16 == (short *) -1) return (0);
+	if (rgb16 == (short *) MAP_FAILED) return (0);
 
 	c = METEOR_CAP_SINGLE ;
         ioctl(i, METEORCAPTUR, &c);