-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Cellular: Fix to delete PDP context on disconnect #10375
Conversation
@AriParkkila, thank you for your changes. |
CI started |
if (_new_context_set) { | ||
delete_current_context(); | ||
} | ||
_cid = -1; |
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.
delete_current_context() sets _cid to -1 only in successful case. What is the meaning of this line?
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.
Disconnect must be set _cid = -1
to not expect any _cid
to be valid/preferred on the next connect request, even when we did not create that _cid
ourselves.
if (_new_context_set) { | ||
delete_current_context(); | ||
} | ||
_cid = -1; |
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.
Same as above
Test run: SUCCESSSummary: 11 of 11 test jobs passed |
@AnttiKauppila Are there any requests for changes left ? |
Test run: SUCCESSSummary: 11 of 11 test jobs passed |
Description
Delete cellular PDP context on
NetworkInterface::disconnect
if one was created while connecting.Pull request type
Reviewers
@jarvte
Release Notes