Changes in HOSTAPD to Support PAC #90
Conversation
|
Below are the associated PRs for this functionality. #88 HLD PR is below we will raise the remaining functionality in new PRs asap. |
|
/azpw run Azure |
|
/AzurePipelines run Azure |
|
No pipelines are associated with this pull request. |
|
@jeff-yin @ridahanif96 pls help review |
|
/azpw run Azure |
|
/AzurePipelines run Azure |
|
No pipelines are associated with this pull request. |
| @@ -176,8 +176,10 @@ static void hostapd_clear_old(struct hostapd_iface *iface) | |||
| * allow them to use the BSS anymore. | |||
| */ | |||
| for (j = 0; j < iface->num_bss; j++) { | |||
| #ifndef CONFIG_SONIC_HOSTAPD | |||
There was a problem hiding this comment.
The design mentions that the hostapd receive packets directly when the mode is configured as MAB. Can you please help with the APIs for that functionality?
There was a problem hiding this comment.
Sorry, I didn't get the question. The hostapd doesn't or never communicates with MAB directly. It communicates only with PACD. Similarly MAB communicates only with PACD. Can you please point to the section in the design document, if it is mentioned.. will correct the same.
There was a problem hiding this comment.
sorry my bad. Somehow I remember seeing that & wanted to check
| @@ -176,8 +176,10 @@ static void hostapd_clear_old(struct hostapd_iface *iface) | |||
| * allow them to use the BSS anymore. | |||
| */ | |||
| for (j = 0; j < iface->num_bss; j++) { | |||
| #ifndef CONFIG_SONIC_HOSTAPD | |||
There was a problem hiding this comment.
sorry my bad. Somehow I remember seeing that & wanted to check
| @@ -33,21 +40,28 @@ | |||
| #pragma pack(push, 1) | |||
| #endif /* _MSC_VER */ | |||
|
|
|||
| #if 0 | |||
There was a problem hiding this comment.
If this code is not required, please remove it.
Why I did it
These changes are required to support PAC functionality to interact with 802.1X.
How I did it
How to verify it