Patch "ALSA: firewire-motu: fix unreleased lock warning in hwdep device" has been added to the 6.1-stable tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This is a note to let you know that I've just added the patch titled

    ALSA: firewire-motu: fix unreleased lock warning in hwdep device

to the 6.1-stable tree which can be found at:
    http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
     alsa-firewire-motu-fix-unreleased-lock-warning-in-hw.patch
and it can be found in the queue-6.1 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@xxxxxxxxxxxxxxx> know about it.



commit bbb1e09de4ccc227f482a35eefd8df63bdb68272
Author: Takashi Sakamoto <o-takashi@xxxxxxxxxxxxx>
Date:   Mon Jan 30 23:15:40 2023 +0900

    ALSA: firewire-motu: fix unreleased lock warning in hwdep device
    
    [ Upstream commit c7a806d9ce6757ff56078674916e53bd859f242d ]
    
    Smatch static analysis tool detects that acquired lock is not released
    in hwdep device when condition branch is passed due to no event. It is
    unlikely to occur, while fulfilling is preferable for better coding.
    
    Reported-by: Dan Carpenter <error27@xxxxxxxxx>
    Fixes: 634ec0b2906e ("ALSA: firewire-motu: notify event for parameter change in register DSP model")
    Signed-off-by: Takashi Sakamoto <o-takashi@xxxxxxxxxxxxx>
    Link: https://lore.kernel.org/r/20230130141540.102854-1-o-takashi@xxxxxxxxxxxxx
    Signed-off-by: Takashi Iwai <tiwai@xxxxxxx>
    Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>

diff --git a/sound/firewire/motu/motu-hwdep.c b/sound/firewire/motu/motu-hwdep.c
index a900fc0e7644..88d1f4b56e4b 100644
--- a/sound/firewire/motu/motu-hwdep.c
+++ b/sound/firewire/motu/motu-hwdep.c
@@ -87,6 +87,10 @@ static long hwdep_read(struct snd_hwdep *hwdep, char __user *buf, long count,
 			return -EFAULT;
 
 		count = consumed;
+	} else {
+		spin_unlock_irq(&motu->lock);
+
+		count = 0;
 	}
 
 	return count;



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux