Skip to content

[fuzzing] Handle closing after the final write in fuzzing-event-engine - #33266

Merged
ctiller merged 1 commit into
grpc:masterfrom
ctiller:nerdy2
Jun 5, 2023
Merged

[fuzzing] Handle closing after the final write in fuzzing-event-engine#33266
ctiller merged 1 commit into
grpc:masterfrom
ctiller:nerdy2

Conversation

@ctiller

@ctiller ctiller commented May 29, 2023

Copy link
Copy Markdown
Member

If an endpoint closes it should still report any pending writes.

@ctiller
ctiller merged commit 72da46f into grpc:master Jun 5, 2023
@copybara-service copybara-service Bot added the imported Specifies if the PR has been imported to the internal repository label Jun 5, 2023
eugeneo pushed a commit to eugeneo/grpc that referenced this pull request Jun 7, 2023
grpc#33266)

If an endpoint closes it should still report any pending writes.
eugeneo pushed a commit to eugeneo/grpc that referenced this pull request Jun 7, 2023
grpc#33266)

If an endpoint closes it should still report any pending writes.
ctiller added a commit that referenced this pull request Jun 14, 2023
… dangling (#33406)

Fix fuzzer found bug b/286716972

Follows up on #33266 but gets the edge
case right of when there's a read queued before the peer closes - in
that case we weren't waking up the read.
mario-vimal pushed a commit to mario-vimal/grpc that referenced this pull request Jun 15, 2023
grpc#33266)

If an endpoint closes it should still report any pending writes.
mario-vimal pushed a commit to mario-vimal/grpc that referenced this pull request Jun 15, 2023
… dangling (grpc#33406)

Fix fuzzer found bug b/286716972

Follows up on grpc#33266 but gets the edge
case right of when there's a read queued before the peer closes - in
that case we weren't waking up the read.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bloat/none imported Specifies if the PR has been imported to the internal repository lang/core per-call-memory/neutral per-channel-memory/neutral release notes: no Indicates if PR should not be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants