anchao commented on PR #7409: URL: https://github.com/apache/incubator-nuttx/pull/7409#issuecomment-1288723427
> Anyway I still think that we are too fast with toolchain upgrade, but I'm more conservative. Do we have a significant gain in optimisation with new compiler? Or what is the rush here? The new toolchain adds more static analyzer on compile stage, which will helpful for code quality: https://gcc.gnu.org/gcc-11/changes.html  For example: ``` if (focus != NULL) { nxipc_client_disconnect(focus->send_proxy); nxipc_sub_disconnect(focus->receive_proxy); free((void*)focus->name); free((void*)focus); pthread_mutex_destroy(&focus->mutex); <-- use-after-free } ``` build will break on gcc-11.3 ([-Werror=use-after-free]) ``` ../common/src/client/app_focus_c.c: In function 'app_focus_release': ../common/src/client/app_focus_c.c:157:9: error: pointer 'handle' used after 'free' [-Werror=use-after-free] 157 | pthread_mutex_destroy(&focus->mutex); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ../common/src/client/app_focus_c.c:156:9: note: call to 'free' here 156 | free((void*)focus); | ^~~~~~~~~~~~~~~~~~ ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org