diff options
author | Russ Cox <rsc@golang.org> | 2014-10-01 16:51:32 -0400 |
---|---|---|
committer | Russ Cox <rsc@golang.org> | 2014-10-01 16:51:32 -0400 |
commit | 4cdb5665c71cab400c38adcaf3297418cabf44d9 (patch) | |
tree | 03c507a6192cfbf1e3e1d9100e4c45623a313ccf | |
parent | 1b379b71a66f387ea1482f24b0fa535f6606433d (diff) | |
download | go-4cdb5665c71cab400c38adcaf3297418cabf44d9.tar.gz |
reflect: fix IsValid vs Kind mismatch after Elem of nil interface
LGTM=r
R=r
CC=golang-codereviews
https://codereview.appspot.com/151960044
-rw-r--r-- | src/reflect/all_test.go | 14 | ||||
-rw-r--r-- | src/reflect/value.go | 4 |
2 files changed, 17 insertions, 1 deletions
diff --git a/src/reflect/all_test.go b/src/reflect/all_test.go index b72c4b176..d17ef5c5e 100644 --- a/src/reflect/all_test.go +++ b/src/reflect/all_test.go @@ -3939,3 +3939,17 @@ func TestValueString(t *testing.T) { t.Errorf("ValueOf(Impl{}).Method(0).String() = %q, want %q", method.String(), "<func() Value>") } } + +func TestInvalid(t *testing.T) { + // Used to have inconsistency between IsValid() and Kind() != Invalid. + type T struct{ v interface{} } + + v := ValueOf(T{}).Field(0) + if v.IsValid() != true || v.Kind() != Interface { + t.Errorf("field: IsValid=%v, Kind=%v, want true, Interface", v.IsValid(), v.Kind()) + } + v = v.Elem() + if v.IsValid() != false || v.Kind() != Invalid { + t.Errorf("field elem: IsValid=%v, Kind=%v, want false, Invalid", v.IsValid(), v.Kind()) + } +} diff --git a/src/reflect/value.go b/src/reflect/value.go index 12d423f3c..9c65ee270 100644 --- a/src/reflect/value.go +++ b/src/reflect/value.go @@ -791,7 +791,9 @@ func (v Value) Elem() Value { })(v.ptr)) } x := unpackEface(eface) - x.flag |= v.flag & flagRO + if x.flag != 0 { + x.flag |= v.flag & flagRO + } return x case Ptr: ptr := v.ptr |