Commit ae12cf6b authored by Guenter Roeck's avatar Guenter Roeck Committed by Greg Kroah-Hartman
Browse files

staging: rtl8188eu: Drop unnecessary NULL check after container_of 



The result of container_of() operations is never NULL unless the extracted
element is the first element of the embedded structure. This is not the
case here. The NULL check is therefore unnecessary and misleading.
Remove it.

This change was made automatically with the following Coccinelle script.

@@
type t;
identifier v;
statement s;
@@

<+...
(
  t v = container_of(...);
|
  v = container_of(...);
)
  ...
  when != v
- if (\( !v \| v == NULL \) ) s
...+>

Signed-off-by: default avatarGuenter Roeck <linux@roeck-us.net>
Link: https://lore.kernel.org/r/20210514165557.1384106-1-linux@roeck-us.net


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent e2a02710

drivers/staging/rtl8188eu/core/rtw_mlme_ext.c

+0 −3
Original line number Diff line number Diff line
@@ -4038,9 +4038,6 @@ void free_mlme_ext_priv(struct mlme_ext_priv *pmlmeext) 
{

	struct adapter *padapter = container_of(pmlmeext, struct adapter, mlmeextpriv);



	if (!padapter)

		return;



	if (padapter->bDriverStopped) {

		del_timer_sync(&pmlmeext->survey_timer);

		del_timer_sync(&pmlmeext->link_timer);