diff options
author | Matti Linnanvuori <mattilinnanvuori@yahoo.com> | 2007-11-05 17:51:02 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-11-05 18:12:33 -0500 |
commit | 01aae97196f2cdfbfebc5a0365bad82d98975588 (patch) | |
tree | f71b836d62dfe36ab9e842e8cedfa719ca63f018 /drivers/telephony/phonedev.c | |
parent | def6ae26a9e69c3e6d0f0054524c76fd32420ecd (diff) |
telephony: phonedev panics if unregistering device not registered [Bug 9266]
Remove panic from phonedev. See
http://bugzilla.kernel.org/show_bug.cgi?id=9266
for details (phonedev panics if unregistering device not registered).
Signed-off-by: Matti Linnanvuori <mattilinnanvuori@yahoo.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/telephony/phonedev.c')
-rw-r--r-- | drivers/telephony/phonedev.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/telephony/phonedev.c b/drivers/telephony/phonedev.c index 4d8c2a5b3297..bcea8d9b718c 100644 --- a/drivers/telephony/phonedev.c +++ b/drivers/telephony/phonedev.c | |||
@@ -120,9 +120,8 @@ int phone_register_device(struct phone_device *p, int unit) | |||
120 | void phone_unregister_device(struct phone_device *pfd) | 120 | void phone_unregister_device(struct phone_device *pfd) |
121 | { | 121 | { |
122 | mutex_lock(&phone_lock); | 122 | mutex_lock(&phone_lock); |
123 | if (phone_device[pfd->minor] != pfd) | 123 | if (likely(phone_device[pfd->minor] == pfd)) |
124 | panic("phone: bad unregister"); | 124 | phone_device[pfd->minor] = NULL; |
125 | phone_device[pfd->minor] = NULL; | ||
126 | mutex_unlock(&phone_lock); | 125 | mutex_unlock(&phone_lock); |
127 | } | 126 | } |
128 | 127 | ||