Skip to content

Fix nil crashes#152

Closed
titanous wants to merge 4 commits intographql-go:masterfrom
titanous:fix-nil-crashes
Closed

Fix nil crashes#152
titanous wants to merge 4 commits intographql-go:masterfrom
titanous:fix-nil-crashes

Conversation

@titanous
Copy link
Contributor

This fixes the rest of the issues I found with go-fuzz.

Closes #148

titanous added 4 commits July 20, 2016 16:57
Nil panic found with go-fuzz.

Signed-off-by: Jonathan Rudenberg <jonathan@titanous.com>
Panic found with go-fuzz.

Signed-off-by: Jonathan Rudenberg <jonathan@titanous.com>
Nil panic found with go-fuzz.

Signed-off-by: Jonathan Rudenberg <jonathan@titanous.com>
Checks for eg (*Interface)(nil).

Signed-off-by: Jonathan Rudenberg <jonathan@titanous.com>
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.06%) to 90.697% when pulling bf92dd2 on titanous:fix-nil-crashes into 491504a on graphql-go:master.

@chris-ramon
Copy link
Member

chris-ramon commented Aug 5, 2017

closing this one in favor of: #223 — thanks a lot @titanous 👍

@chris-ramon chris-ramon closed this Aug 5, 2017
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.

nil panic in FieldsOnCorrectTypeRule

3 participants