RE: [EXT] Re: [bug report] wireless: mwifiex: initial commit for Marvell mwifiex driver

[Ganapathi Bhat <ganapathi.bhat@xxxxxxx>]

Hi Dan/Brian

>    495  void mwifiex_11n_delete_tx_ba_stream_tbl_entry(struct
> mwifiex_private *priv,
>    496                                  struct mwifiex_tx_ba_stream_tbl *tx_ba_tsr_tbl)
>    497  {
>    498          if (!tx_ba_tsr_tbl &&
>                      ^^^^^^^^^^^^^
> Check for NULL
> 
>    499              mwifiex_is_tx_ba_stream_ptr_valid(priv, tx_ba_tsr_tbl))
>                                                             ^^^^^^^^^^^^^ Which is passed to here.  So
> maybe the NULL check is reversed?

I think, it should have been like below: 

if (!tx_ba_tsr_tbl || !mwifiex_is_tx_ba_stream_ptr_valid(priv, tx_ba_tsr_tbl)) . . . 


Regards,
Ganapathi