Commit 3b371fe4 authored Jan 03, 2014 by Dan Carpenter Committed by Greg Kroah-Hartman Jan 08, 2014

staging: vt6656: remove an unneeded NULL check



We dereference "param->u.wpa_key.key" on the next line so the check
here is inconsistent.  This is only called from iwctl_siwencodeext() and
"param->u.wpa_key.key" is a valid pointer.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent 1c5ebaae

drivers/staging/vt6656/wpactl.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -86,7 +86,7 @@ int wpa_set_keys(struct vnt_private pDevice, void ctx)
		return ret;
		}

		if (param->u.wpa_key.key && param->u.wpa_key.key_len > sizeof(abyKey))
		if (param->u.wpa_key.key_len > sizeof(abyKey))
		return -EINVAL;

		memcpy(&abyKey[0], param->u.wpa_key.key, param->u.wpa_key.key_len);