From: Jose Alonso <joalonsof@gmail.com>
Date: Sun, 10 Jul 2011 18:46:51 +0000 (-0300)
Subject: dell-laptop - using buffer without mutex_lock
X-Git-Url: http://git.osdn.net/view?a=commitdiff_plain;h=b486742a12a474a01d1acb1a5924af11d9b32b68;p=sagit-ice-cold%2Fkernel_xiaomi_msm8998.git

dell-laptop - using buffer without mutex_lock

Using buffer->output[1] without mutex_lock()

Signed-off-by: Jose Alonso <joalonsof@gmail.com>
Signed-off-by: Matthew Garrett <mjg@redhat.com>
---

diff --git a/drivers/platform/x86/dell-laptop.c b/drivers/platform/x86/dell-laptop.c
index 04c34f1a265c..e39ab1d3ed87 100644
--- a/drivers/platform/x86/dell-laptop.c
+++ b/drivers/platform/x86/dell-laptop.c
@@ -540,11 +540,11 @@ static int dell_get_intensity(struct backlight_device *bd)
 	else
 		dell_send_request(buffer, 0, 1);
 
+	ret = buffer->output[1];
+
 out:
 	release_buffer();
-	if (ret)
-		return ret;
-	return buffer->output[1];
+	return ret;
 }
 
 static const struct backlight_ops dell_ops = {