Skip to content
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

runtime: fix stack split at bad time when fuzzing #53191

Conversation

kyakdan
Copy link
Contributor

@kyakdan kyakdan commented Jun 1, 2022

Fix #53190

@gopherbot
Copy link

This PR (HEAD: 2e05bd9) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/go/+/410034 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@kyakdan kyakdan force-pushed the fix-bad-stack-split-when-fuzzing branch from 2e05bd9 to 934d450 Compare June 1, 2022 22:30
@gopherbot
Copy link

This PR (HEAD: 934d450) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/go/+/410034 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@gopherbot
Copy link

Message from Michael Knyszek:

Patch Set 2:

(1 comment)


Please don’t reply on this GitHub thread. Visit golang.org/cl/410034.
After addressing review feedback, remember to publish your drafts!

@kyakdan kyakdan force-pushed the fix-bad-stack-split-when-fuzzing branch from 934d450 to bedbd5e Compare June 3, 2022 14:03
@gopherbot
Copy link

This PR (HEAD: bedbd5e) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/go/+/410034 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@gopherbot
Copy link

Message from Khaled Yakdan:

Patch Set 3:

(1 comment)


Please don’t reply on this GitHub thread. Visit golang.org/cl/410034.
After addressing review feedback, remember to publish your drafts!

@gopherbot
Copy link

Message from Michael Knyszek:

Patch Set 3: Run-TryBot+1 Code-Review+2

(1 comment)


Please don’t reply on this GitHub thread. Visit golang.org/cl/410034.
After addressing review feedback, remember to publish your drafts!

@gopherbot
Copy link

Message from Gopher Robot:

Patch Set 3:

(1 comment)


Please don’t reply on this GitHub thread. Visit golang.org/cl/410034.
After addressing review feedback, remember to publish your drafts!

@gopherbot
Copy link

Message from Gopher Robot:

Patch Set 3: TryBot-Result+1

(1 comment)


Please don’t reply on this GitHub thread. Visit golang.org/cl/410034.
After addressing review feedback, remember to publish your drafts!

@gopherbot
Copy link

Message from Dmitri Shuralyov:

Patch Set 3: Code-Review+1


Please don’t reply on this GitHub thread. Visit golang.org/cl/410034.
After addressing review feedback, remember to publish your drafts!

@kyakdan kyakdan force-pushed the fix-bad-stack-split-when-fuzzing branch from bedbd5e to 5b260b3 Compare June 9, 2022 07:23
@gopherbot
Copy link

This PR (HEAD: 5b260b3) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/go/+/410034 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@kyakdan kyakdan force-pushed the fix-bad-stack-split-when-fuzzing branch from 5b260b3 to 13b0749 Compare June 9, 2022 07:28
@gopherbot
Copy link

This PR (HEAD: 13b0749) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/go/+/410034 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

gopherbot pushed a commit that referenced this pull request Jun 10, 2022
Fix #53190

Change-Id: I6c1f9c3ab58818d3a9f05ddaa02fc247e53677d3
GitHub-Last-Rev: 13b0749
GitHub-Pull-Request: #53191
Reviewed-on: https://go-review.googlesource.com/c/go/+/410034
Reviewed-by: Michael Knyszek <mknyszek@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
@gopherbot
Copy link

This PR is being closed because golang.org/cl/410034 has been merged.

@gopherbot gopherbot closed this Jun 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

runtime: stack split at bad time during fuzzing
2 participants