Messages in this thread Patch in this message | | | Date | Fri, 11 Jul 2014 19:22:04 +0900 | From | Daeseok Youn <> | Subject | [PATCH 1/9] staging: dgap: redundant NULL and magic check in dgap_get_modem_info() |
| |
The "ch" is already checking in caller.
Signed-off-by: Daeseok Youn <daeseok.youn@gmail.com> --- drivers/staging/dgap/dgap.c | 3 --- 1 files changed, 0 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/dgap/dgap.c b/drivers/staging/dgap/dgap.c index 9affc5b..79ea116 100644 --- a/drivers/staging/dgap/dgap.c +++ b/drivers/staging/dgap/dgap.c @@ -3061,9 +3061,6 @@ static int dgap_get_modem_info(struct channel_t *ch, unsigned int __user *value) ulong lock_flags; int rc; - if (!ch || ch->magic != DGAP_CHANNEL_MAGIC) - return -EIO; - spin_lock_irqsave(&ch->ch_lock, lock_flags); mstat = readb(&(ch->ch_bs->m_stat)); -- 1.7.1
| |