summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDaeseok Youn <daeseok.youn@gmail.com>2014-04-16 18:48:02 +0900
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2014-04-16 14:03:40 -0700
commitf7a87195603ae9db133ddd393169dd6c1d45b4a3 (patch)
treef060b041e2155815afa96448022fbd7191862320
parent895d240d1db0b2736d779200788e4c4aea28a0c6 (diff)
downloadop-kernel-dev-f7a87195603ae9db133ddd393169dd6c1d45b4a3.zip
op-kernel-dev-f7a87195603ae9db133ddd393169dd6c1d45b4a3.tar.gz
uwb: adds missing error handling
There is checking NULL before dereferncing but it need to add "return". Signed-off-by: Daeseok Youn <daeseok.youn@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r--drivers/uwb/drp.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/uwb/drp.c b/drivers/uwb/drp.c
index 16ada83..1a2fd97 100644
--- a/drivers/uwb/drp.c
+++ b/drivers/uwb/drp.c
@@ -599,8 +599,11 @@ static void uwb_drp_handle_alien_drp(struct uwb_rc *rc, struct uwb_ie_drp *drp_i
/* alloc and initialize new uwb_cnflt_alien */
cnflt = kzalloc(sizeof(struct uwb_cnflt_alien), GFP_KERNEL);
- if (!cnflt)
+ if (!cnflt) {
dev_err(dev, "failed to alloc uwb_cnflt_alien struct\n");
+ return;
+ }
+
INIT_LIST_HEAD(&cnflt->rc_node);
init_timer(&cnflt->timer);
cnflt->timer.function = uwb_cnflt_timer;
OpenPOWER on IntegriCloud