[patch] [media] rc: double unlock in rc_register_device()

2011-05-26 Thread Dan Carpenter
If change_protocol() fails and we goto out_raw, then it calls unlock twice. I noticed that the other time we called change_protocol() we held the dev-lock, so I changed it to hold it here too. Signed-off-by: Dan Carpenter erro...@gmail.com --- Compile tested only. diff --git

Re: [patch] [media] rc: double unlock in rc_register_device()

2011-05-26 Thread Jarod Wilson
Dan Carpenter wrote: If change_protocol() fails and we goto out_raw, then it calls unlock twice. Gah, good catch, I gotta quit adding new bugs... ;) I noticed that the other time we called change_protocol() we held the dev-lock, so I changed it to hold it here too. Signed-off-by: Dan