diff options
| author | Dan Carpenter <error27@gmail.com> | 2009-11-29 01:02:55 -0500 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2009-11-30 03:06:51 -0500 |
| commit | 40be261dfd8bfba4baeff40168d44a6a4450ace1 (patch) | |
| tree | 8ac8771778d4fbfaa8a1973bccaaea5dc8e1604b | |
| parent | 3c91c7ae84966f992429e1e128c4936f22b89064 (diff) | |
wan: cosa: drop chan->wsem on error path
The other paths all drop chan->wsem. This was found by a static
checker (smatch).
Signed-off-by: Dan Carpenter <error27@gmail.com>
Acked-by: Jan "Yenya" Kasprzak <kas@fi.muni.cz>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/net/wan/cosa.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/net/wan/cosa.c b/drivers/net/wan/cosa.c index e2c33c06190b..8e25ca7080c7 100644 --- a/drivers/net/wan/cosa.c +++ b/drivers/net/wan/cosa.c | |||
| @@ -907,6 +907,7 @@ static ssize_t cosa_write(struct file *file, | |||
| 907 | current->state = TASK_RUNNING; | 907 | current->state = TASK_RUNNING; |
| 908 | chan->tx_status = 1; | 908 | chan->tx_status = 1; |
| 909 | spin_unlock_irqrestore(&cosa->lock, flags); | 909 | spin_unlock_irqrestore(&cosa->lock, flags); |
| 910 | up(&chan->wsem); | ||
| 910 | return -ERESTARTSYS; | 911 | return -ERESTARTSYS; |
| 911 | } | 912 | } |
| 912 | } | 913 | } |
