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)) > _______________________________________________ svn-src-all@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-all To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"
