diff options
author | Gustavo A. R. Silva <garsilva@embeddedor.com> | 2017-06-13 12:37:18 -0400 |
---|---|---|
committer | Samuel Ortiz <sameo@linux.intel.com> | 2017-06-22 18:34:18 -0400 |
commit | 03036184e9d4a5b2b42a70b66db9455808dd5da9 (patch) | |
tree | 7cae87bbf374c9102acb636c0cdc97a25e40e9ff /net/nfc | |
parent | a0323b979f81ad2deb2c8836eab506534891876a (diff) |
nfc: nci: remove unnecessary null check
Remove unnecessary NULL check for pointer conn_info.
conn_info is set in list_for_each_entry() using container_of(),
which is never NULL.
Addresses-Coverity-ID: 1362349
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Gustavo A. R. Silva <garsilva@embeddedor.com>
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
Diffstat (limited to 'net/nfc')
-rw-r--r-- | net/nfc/nci/core.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/net/nfc/nci/core.c b/net/nfc/nci/core.c index 9ec8d8736378..c25e9b4179c3 100644 --- a/net/nfc/nci/core.c +++ b/net/nfc/nci/core.c | |||
@@ -73,11 +73,10 @@ int nci_get_conn_info_by_dest_type_params(struct nci_dev *ndev, u8 dest_type, | |||
73 | if (conn_info->dest_type == dest_type) { | 73 | if (conn_info->dest_type == dest_type) { |
74 | if (!params) | 74 | if (!params) |
75 | return conn_info->conn_id; | 75 | return conn_info->conn_id; |
76 | if (conn_info) { | 76 | |
77 | if (params->id == conn_info->dest_params->id && | 77 | if (params->id == conn_info->dest_params->id && |
78 | params->protocol == conn_info->dest_params->protocol) | 78 | params->protocol == conn_info->dest_params->protocol) |
79 | return conn_info->conn_id; | 79 | return conn_info->conn_id; |
80 | } | ||
81 | } | 80 | } |
82 | } | 81 | } |
83 | 82 | ||