This patch doesn't apply to the .36 stable tree
If someone wants it applied there, please email the backport
to [email protected]

thanks,

greg k-h

> commit: 136711be41ec97f7f1a9c3a5e8535eb7da5fea59
> From: Takashi Iwai <[email protected]>
> Date: Sat, 4 Dec 2010 16:13:06 +0100
> Subject: [PATCH] agp/intel: Fix wrong kunmap in i830_cleanup()
> 
> Add a missing NULL check and fix the wrong address passed to kunmap()
> in i830_cleanup().
> 
> Cc: [email protected]
> Signed-off-by: Takashi Iwai <[email protected]>
> [danvet: added cc stable]
> Signed-off-by: Daniel Vetter <[email protected]>
> 
> Signed-off-by: Chris Wilson <[email protected]>
> ---
>  drivers/char/agp/intel-gtt.c |    6 ++++--
>  1 files changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/char/agp/intel-gtt.c b/drivers/char/agp/intel-gtt.c
> index 9272c38..16a2847 100644
> --- a/drivers/char/agp/intel-gtt.c
> +++ b/drivers/char/agp/intel-gtt.c
> @@ -812,8 +812,10 @@ static int intel_fake_agp_fetch_size(void)
>  
>  static void i830_cleanup(void)
>  {
> -     kunmap(intel_private.i8xx_page);
> -     intel_private.i8xx_flush_page = NULL;
> +     if (intel_private.i8xx_flush_page) {
> +             kunmap(intel_private.i8xx_flush_page);
> +             intel_private.i8xx_flush_page = NULL;
> +     }
>  
>       __free_page(intel_private.i8xx_page);
>       intel_private.i8xx_page = NULL;
> 
> _______________________________________________
> stable mailing list
> [email protected]
> http://linux.kernel.org/mailman/listinfo/stable

_______________________________________________
stable mailing list
[email protected]
http://linux.kernel.org/mailman/listinfo/stable

Reply via email to