Skip to content

check slice size#282

Open
FaKod wants to merge 1 commit intoupmc-enterprises:masterfrom
FaKod:fix-slice-panic
Open

check slice size#282
FaKod wants to merge 1 commit intoupmc-enterprises:masterfrom
FaKod:fix-slice-panic

Conversation

@FaKod
Copy link
Copy Markdown
Contributor

@FaKod FaKod commented Feb 8, 2019

avoids crash mentioned in #267

@FaKod FaKod mentioned this pull request Feb 8, 2019
@vroudge
Copy link
Copy Markdown

vroudge commented Feb 21, 2019

Can we get this passed? I'm getting this issue in production right now.

@FaKod
Copy link
Copy Markdown
Contributor Author

FaKod commented Mar 29, 2019

@stevesloka Can we have this patch (or a similar one) in a release?

Copy link
Copy Markdown

@abh abh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me ...

@abh
Copy link
Copy Markdown

abh commented May 4, 2019

oh, it was fixed from #303 (so this can be closed I think).

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.

3 participants