Skip to content
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

wpa_supplicant: Fix memory leak in esp_issue_scan error paths (IDFGH-5510) #7240

Closed

Conversation

AxelLin
Copy link
Contributor

@AxelLin AxelLin commented Jul 7, 2021

Fix memory leak when allocate memory for params->ssid / params->bssid fails.

Fixes: 27101f9 ("wpa_supplicant: Add initial roaming support")
Signed-off-by: Axel Lin axel.lin@gmail.com

Fix memory leak when allocate memory for params->ssid / params->bssid fails.

Fixes: 27101f9 ("wpa_supplicant: Add initial roaming support")
Signed-off-by: Axel Lin <axel.lin@gmail.com>
@AxelLin
Copy link
Contributor Author

AxelLin commented Jul 7, 2021

Add @kapilkedawat for review.

@espressif-bot espressif-bot added the Status: Opened Issue is new label Jul 7, 2021
@github-actions github-actions bot changed the title wpa_supplicant: Fix memory leak in esp_issue_scan error paths wpa_supplicant: Fix memory leak in esp_issue_scan error paths (IDFGH-5510) Jul 7, 2021
@AxelLin
Copy link
Contributor Author

AxelLin commented Jul 28, 2021

@kapilkedawat @Alvin1Zhang

Sigh, there is no response so far.
I'm not sure what can I do now.
Should I create an issue instead?

@sagb2015
Copy link
Contributor

@AxelLin Thanks for your contribution! We will merge this internally soon.

@espressif-bot espressif-bot added Status: Done Issue is done internally Resolution: Done Issue is done internally and removed Status: Opened Issue is new labels Aug 11, 2021
espressif-bot pushed a commit that referenced this pull request Aug 11, 2021
Fix memory leak when allocate memory for params->ssid / params->bssid fails.

Fixes: 27101f9 ("wpa_supplicant: Add initial roaming support")
Signed-off-by: Axel Lin <axel.lin@gmail.com>
Signed-off-by: Sagar Bijwe <sagar@espressif.com>

Merges #7240
@Alvin1Zhang
Copy link
Collaborator

Thanks for your contribution again, changes have been merged with e144a60.

@AxelLin AxelLin deleted the esp_issue_scan_memory_leak_fix branch August 12, 2021 01:28
espressif-bot pushed a commit that referenced this pull request Aug 15, 2021
Fix memory leak when allocate memory for params->ssid / params->bssid fails.

Fixes: 27101f9 ("wpa_supplicant: Add initial roaming support")
Signed-off-by: Axel Lin <axel.lin@gmail.com>
Signed-off-by: Sagar Bijwe <sagar@espressif.com>

Merges #7240
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants