Another bug in the same function:
One character difference kills the meshd-nl80211
static int handle_del_peer(struct netlink_config_s *nlcfg,
struct nl_msg *msg, void *arg)
{
...
if ((peer = find_peer(nla_data(tb[NL80211_ATTR_MAX]), 0))) //SHOULD
BE NL80211_ATTR_MAC
delete_peer(&peer);
...
return 0;
}
-----Original Message-----
From: [email protected]
[mailto:[email protected]] On Behalf Of Chaoxing Lin
Sent: Tuesday, April 09, 2013 1:51 PM
To: [email protected]
Subject: RE: authsae bug in handle_del_peer()
After remove ";" in line 357, more problem in this function handle_del_peer()
It can't compile well.
The "struct candidate **peer;" should be changed to "struct candidate *peer;"
And all references to peer in this function should be changed accordingly.
-----Original Message-----
From: [email protected]
[mailto:[email protected]] On Behalf Of Chaoxing Lin
Sent: Tuesday, April 09, 2013 1:43 PM
To: [email protected]
Subject: authsae bug in handle_del_peer()
File: linux/meshd-nl80211.c
Line 357~358:
357 if (nla_get_u32(tb[NL80211_ATTR_IFINDEX]) != nlcfg->ifindex);
358 return -1;
The ";" at the end of line 357 is the problem.
The impact is severe.
handle_del_peer() always return -1 prematurelly.
Meshd-nl80211 can not handle station delete event from kernel.
_______________________________________________
Devel mailing list
[email protected]
http://lists.open80211s.org/cgi-bin/mailman/listinfo/devel
_______________________________________________
Devel mailing list
[email protected]
http://lists.open80211s.org/cgi-bin/mailman/listinfo/devel
_______________________________________________
Devel mailing list
[email protected]
http://lists.open80211s.org/cgi-bin/mailman/listinfo/devel
