-
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: fixed crash when closing CellularNetwork via CellularDevice. #8019
Cellular: fixed crash when closing CellularNetwork via CellularDevice. #8019
Conversation
@@ -29,7 +29,7 @@ namespace mbed { | |||
class AT_CellularBase { | |||
public: | |||
AT_CellularBase(ATHandler &at); | |||
|
|||
virtual ~AT_CellularBase() {} |
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.
Strictly not needed due to the class has no virtual members, but I guess this is a good precaution.
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.
how is this used (looking at the class, there's no virtual method here), that we need to have dtor virtual?
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.
removed virtual dtor
787c144
to
4ac29e9
Compare
4ac29e9
to
ed7606b
Compare
/morph build |
Build : SUCCESSBuild number : 3139 Triggering tests/morph test |
Exporter Build : SUCCESSBuild number : 2746 |
Test : SUCCESSBuild number : 2943 |
Based on top of changes going into 5.11.0-rc1 (#7969) |
Description
ATHandler was released before network instance was deleted and in network destructor athandler was used.
Internal ref to defect: IOTCELL-1321
@AriParkkila please review
Pull request type