lkml.org 
[lkml]   [2005]   [Jan]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
SubjectRE: [PATCH][3/4] let's kill verify_area - convert kernel/compat.c to access_ok()
Date
From
> -----Original Message-----
> From: linux-kernel-owner@vger.kernel.org
[mailto:linux-kernel-owner@vger.kernel.org] On Behalf Of Jesper Juhl
> Sent: Friday, January 07, 2005 02:19
> To: linux-kernel
> Cc: Andrew Morton
> Subject: [PATCH][3/4] let's kill verify_area - convert kernel/compat.c
to access_ok()
>
> Here's a patch to convert verify_area to access_ok in kernel/compat.c
>
> diff -up linux-2.6.10-bk9-orig/kernel/compat.c
> linux-2.6.10-bk9/kernel/compat.c
> --- linux-2.6.10-bk9-orig/kernel/compat.c 2005-01-06
22:19:13.000000000 +0100
> +++ linux-2.6.10-bk9/kernel/compat.c 2005-01-07 02:06:00.000000000
+0100

[snip]

> @@ -612,7 +612,7 @@ long compat_get_bitmap(unsigned long *ma
> /* align bitmap up to nearest compat_long_t boundary */
> bitmap_size = ALIGN(bitmap_size, BITS_PER_COMPAT_LONG);
>
> - if (verify_area(VERIFY_READ, umask, bitmap_size / 8))
> + if (!access_ok(VERIFY_READ, umask, bitmap_size / 8) != 0)

Please do not use double negations (i.e., drop the '!= 0' test
again).

> return -EFAULT;
>
> nr_compat_longs = BITS_TO_COMPAT_LONGS(bitmap_size);
> @@ -653,7 +653,7 @@ long compat_put_bitmap(compat_ulong_t __
> /* align bitmap up to nearest compat_long_t boundary */
> bitmap_size = ALIGN(bitmap_size, BITS_PER_COMPAT_LONG);
>
> - if (verify_area(VERIFY_WRITE, umask, bitmap_size / 8))
> + if (!access_ok(VERIFY_WRITE, umask, bitmap_size / 8) != 0)
> return -EFAULT;
>
> nr_compat_longs = BITS_TO_COMPAT_LONGS(bitmap_size);

//Peter
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 14:09    [W:0.064 / U:0.704 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site