-
Notifications
You must be signed in to change notification settings - Fork 3k
Cellular: fixed defect where SIM interface was closed too early #7275
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
Conversation
…ng crash with certain configurations.
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.
Please push also to mbed-os-5.9
@jarvte @AriParkkila We're a bit confused. As is, this PR look like it will be going into 5.9.2. |
Copy edit for consistent tense and comma usage.
/morph build |
Build : SUCCESSBuild number : 2422 Triggering tests/morph test |
Exporter Build : SUCCESSBuild number : 2052 |
@cmonr this fix should be good for all 5.9, 5.10 and master. |
Description
SIM interface was closed too early in CellularConnectionFSM.cpp causing a crash if APN lookup was enabled and apn configured to '0' in application json.
Crash happened EasyCellularConnection::connect(). Moved SIM interface closing a bit later.
Internal ref to defect: IOTCELL-1122
@AriParkkila please review
Pull request type