-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HOSTPAD driver changes for PAC #91
base: master
Are you sure you want to change the base?
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 |
Ali Hussain from xFlow will review these PRs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes seems fine. Please rerun of push again the code to pass the failed checks
fprintf(stderr, "socket failed [%s]\n", strerror(errno)); | ||
return -1; | ||
} | ||
printf("Created a socket with fd: %d\n", fd); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need printfs in this function?
Why I did it
These changes are required to support PAC functionality to sonic-wpasupplicant driver.
How I did it
How to verify it