svn commit: r215612 - stable/8/sys/net80211

[Stefan Farfeleder]

On Sun, Nov 21, 2010 at 10:49:41AM +0000, Bernhard Schmidt wrote:
> Author: bschmidt
> Date: Sun Nov 21 10:49:40 2010
> New Revision: 215612
> URL: http://svn.freebsd.org/changeset/base/215612
> 
> Log:
>   Compare the address of the array, not the array.
>   
>   Committed by:	rpaulo
>   Found with:	Coverity Prevent(tm)
>   CID:		3690

This doesn't make sense to me. How can the address of the array member
be null? Also comparing the array and comparing its address has the same
effect as the array is converted to the pointer anyway in the context of
the comparison.

Stefan

> Modified:
>   stable/8/sys/net80211/ieee80211_scan_sta.c
> Directory Properties:
>   stable/8/sys/   (props changed)
>   stable/8/sys/amd64/include/xen/   (props changed)
>   stable/8/sys/cddl/contrib/opensolaris/   (props changed)
>   stable/8/sys/contrib/dev/acpica/   (props changed)
>   stable/8/sys/contrib/pf/   (props changed)
>   stable/8/sys/dev/xen/xenpci/   (props changed)
> 
> Modified: stable/8/sys/net80211/ieee80211_scan_sta.c
> ==============================================================================
> --- stable/8/sys/net80211/ieee80211_scan_sta.c	Sun Nov 21 10:45:10 2010	(r215611)
> +++ stable/8/sys/net80211/ieee80211_scan_sta.c	Sun Nov 21 10:49:40 2010	(r215612)
> @@ -1013,7 +1013,7 @@ match_bss(struct ieee80211vap *vap,
>  		 */
>  		if (se->se_capinfo & (IEEE80211_CAPINFO_IBSS|IEEE80211_CAPINFO_ESS))
>  			fail |= MATCH_CAPINFO;
> -		else if (se->se_meshid == NULL)
> +		else if (&se->se_meshid == NULL)
>  			fail |= MATCH_MESH_NOID;
>  		else if (ms->ms_idlen != 0 &&
>  		    match_id(se->se_meshid, ms->ms_id, ms->ms_idlen))
>