Skip to content

Conversation

Lsitar
Copy link

@Lsitar Lsitar commented Jun 30, 2022

Pointer to gpio_conf variable was used out of its scope. This caused dereference to some trash values in the following call usb_new_phy(). I moved the variable definition to include usb_new_phy() call in its scope and now it's working ok.

@CLAassistant
Copy link

CLAassistant commented Jun 30, 2022

CLA assistant check
All committers have signed the CLA.

@espressif-bot espressif-bot added the Status: Opened Issue is new label Jun 30, 2022
@github-actions github-actions bot changed the title Fix tinyusb_driver_install (dangling pointer) Fix tinyusb_driver_install (dangling pointer) (IDFGH-7723) Jun 30, 2022
@Alvin1Zhang
Copy link
Collaborator

Thanks for your contribution.

@suda-morris suda-morris added the PR-Sync-Merge Pull request sync as merge commit label Jul 5, 2022
@suda-morris
Copy link
Collaborator

sha=db90b512a11479a11275b4f08d5ea35127d9c4c0

@espressif-bot espressif-bot added Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally Resolution: Done Issue is done internally and removed Status: Opened Issue is new Resolution: NA Issue resolution is unavailable labels Jul 6, 2022
@espressif-bot espressif-bot merged commit 8b30b7a into espressif:master Jul 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR-Sync-Merge Pull request sync as merge commit Resolution: Done Issue is done internally Status: Done Issue is done internally
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants