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

Fixed rare crash of DHCP Server during WIFI AP network reconfiguratio… (IDFGH-1151) #3466

Conversation

despierto
Copy link

…n and DHCP restart due to missed callback

There are few places of dhcps_start(), but at one of them doesn't have initialization of dhcps_cb by dhcps_set_new_lease_cb().

Unexpected condition while network reconfiguration with calling of dhcps_start() from another place happened a few times and caused to system crash by reading NULL ptr.

@CLAassistant
Copy link

CLAassistant commented May 13, 2019

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions github-actions bot changed the title Fixed rare crash of DHCP Server during WIFI AP network reconfiguratio… Fixed rare crash of DHCP Server during WIFI AP network reconfiguratio… (IDFGH-1151) May 13, 2019
@Alvin1Zhang
Copy link
Collaborator

@despierto Thanks for the contribution and sorry for the slow turnaround. We will look into this. Thanks.

@freakyxue
Copy link
Collaborator

@despierto
this bug has been fixed in 3.3.
later i will backport to 3.2

@AxelLin
Copy link
Contributor

AxelLin commented Dec 16, 2020

v3.3: the fix is merged by faf23c0
v3.2: the fix is included in 49202b7
So this should be closed.

@Alvin1Zhang
Copy link
Collaborator

@despierto Thanks for contribution, and thanks @AxelLin for the notes, changes have been merged with faf23c0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants