diff options
author | Emmanuel Odeke <emm.odeke@gmail.com> | 2016-07-17 14:22:15 -0700 |
---|---|---|
committer | Rob Pike <r@golang.org> | 2016-08-19 21:54:39 +0000 |
commit | 0a2a64d85d52ad51de34d39bc5685c39c0e1e32a (patch) | |
tree | c42b4fe4ab44670f006550aaa27a4aff52910eee /src/encoding/gob | |
parent | 14e59511661303ab1406f7c21ee27e58bcd0750e (diff) | |
download | go-0a2a64d85d52ad51de34d39bc5685c39c0e1e32a.tar.gz go-0a2a64d85d52ad51de34d39bc5685c39c0e1e32a.zip |
encoding/gob: error out instead of panicking on nil dereference
Do not panic when we encounter nil interface values which are
invalid values for gob. Previously this wasn't caught yet
we were calling reflect.*.Type() on reflect.Invalid values
thereby causing panic:
`panic: reflect: call of reflect.Value.Type on zero Value.`
which is a panic not enforced by encoding/gob itself.
We can catch this and send back an error to the caller.
Fixes #16204
Change-Id: Ie646796db297759a74a02eee5267713adbe0c3a0
Reviewed-on: https://go-review.googlesource.com/24989
Reviewed-by: Rob Pike <r@golang.org>
Run-TryBot: Rob Pike <r@golang.org>
Diffstat (limited to 'src/encoding/gob')
-rw-r--r-- | src/encoding/gob/encoder.go | 3 | ||||
-rw-r--r-- | src/encoding/gob/encoder_test.go | 14 |
2 files changed, 17 insertions, 0 deletions
diff --git a/src/encoding/gob/encoder.go b/src/encoding/gob/encoder.go index d6c8fdd963..40ec81b6e6 100644 --- a/src/encoding/gob/encoder.go +++ b/src/encoding/gob/encoder.go @@ -215,6 +215,9 @@ func (enc *Encoder) sendTypeId(state *encoderState, ut *userTypeInfo) { // guaranteeing that all necessary type information has been transmitted first. // Passing a nil pointer to EncodeValue will panic, as they cannot be transmitted by gob. func (enc *Encoder) EncodeValue(value reflect.Value) error { + if value.Kind() == reflect.Invalid { + return errors.New("gob: cannot encode nil value") + } if value.Kind() == reflect.Ptr && value.IsNil() { panic("gob: cannot encode nil pointer of type " + value.Type().String()) } diff --git a/src/encoding/gob/encoder_test.go b/src/encoding/gob/encoder_test.go index 22090a18a6..9256848b50 100644 --- a/src/encoding/gob/encoder_test.go +++ b/src/encoding/gob/encoder_test.go @@ -830,6 +830,20 @@ func TestPtrToMapOfMap(t *testing.T) { } } +// Test that untyped nils generate an error, not a panic. +// See Issue 16204. +func TestCatchInvalidNilValue(t *testing.T) { + encodeErr, panicErr := encodeAndRecover(nil) + if panicErr != nil { + t.Fatalf("panicErr=%v, should not panic encoding untyped nil", panicErr) + } + if encodeErr == nil { + t.Errorf("got err=nil, want non-nil error when encoding untyped nil value") + } else if !strings.Contains(encodeErr.Error(), "nil value") { + t.Errorf("expected 'nil value' error; got err=%v", encodeErr) + } +} + // A top-level nil pointer generates a panic with a helpful string-valued message. func TestTopLevelNilPointer(t *testing.T) { var ip *int |