diff options
| author | Jose Alonso <joalonsof@gmail.com> | 2011-07-10 15:46:51 -0300 |
|---|---|---|
| committer | Matthew Garrett <mjg@redhat.com> | 2011-07-11 09:52:31 -0400 |
| commit | b486742a12a474a01d1acb1a5924af11d9b32b68 (patch) | |
| tree | e2ea6e330e16219b60a349e7c5cbbf5ef829729d /drivers/platform/x86 | |
| parent | be65dde82a4a402e9607c2f306f343bf0912623c (diff) | |
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>
Diffstat (limited to 'drivers/platform/x86')
| -rw-r--r-- | drivers/platform/x86/dell-laptop.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/platform/x86/dell-laptop.c b/drivers/platform/x86/dell-laptop.c index 04c34f1a265..e39ab1d3ed8 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 = { |