Skip to content

close sockfd before set -1 #73

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Dec 1, 2016
Merged

close sockfd before set -1 #73

merged 1 commit into from
Dec 1, 2016

Conversation

hirotakaster
Copy link
Contributor

Hi, it's needed close sockfd before sockfd = -1, and

_connected = false;
close(sockfd);
sockfd = -1;

this equals stop() function.

@me-no-dev
Copy link
Member

Great! I suppose you have tested this?

@hirotakaster
Copy link
Contributor Author

Hi, I compile & test it. But if you okay, please test this part for double check.

@me-no-dev me-no-dev merged commit 3c81739 into espressif:master Dec 1, 2016
@me-no-dev
Copy link
Member

Thanks!

@nomis nomis mentioned this pull request Jul 3, 2022
Closed
1 task
Lzw655 pushed a commit to Lzw655/arduino-esp32 that referenced this pull request Oct 12, 2023
@nomis
Enable BOOTLOADER_APP_ROLLBACK (espressif#73)
19c6911 
Fixes espressif#6936
brentru pushed a commit to adafruit/arduino-esp32 that referenced this pull request Oct 22, 2024
@siddacious
Merge pull request espressif#73 from adafruit/docs_badge
37dc6a1 
Added doxygen badge to assets
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hirotakaster @me-no-dev