diff options
| author | Jose Alonso <joalonsof@gmail.com> | 2011-07-10 14:46:51 -0400 |
|---|---|---|
| committer | Matthew Garrett <mjg@redhat.com> | 2011-07-11 09:52:31 -0400 |
| commit | b486742a12a474a01d1acb1a5924af11d9b32b68 (patch) | |
| tree | e2ea6e330e16219b60a349e7c5cbbf5ef829729d | |
| 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>
| -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 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) | |||
| 540 | else | 540 | else |
| 541 | dell_send_request(buffer, 0, 1); | 541 | dell_send_request(buffer, 0, 1); |
| 542 | 542 | ||
| 543 | ret = buffer->output[1]; | ||
| 544 | |||
| 543 | out: | 545 | out: |
| 544 | release_buffer(); | 546 | release_buffer(); |
| 545 | if (ret) | 547 | return ret; |
| 546 | return ret; | ||
| 547 | return buffer->output[1]; | ||
| 548 | } | 548 | } |
| 549 | 549 | ||
| 550 | static const struct backlight_ops dell_ops = { | 550 | static const struct backlight_ops dell_ops = { |
