reflect: Error if Value.Addr().Elem.CanSet() diff from Value.CanSet()

Author: kezhuw

src/reflect/all_test.go

@@ -350,6 +350,7 @@ func TestCanSetField(t *testing.T) {
 	}
 
 	type testCase struct {
+		// -1 means taking Addr() of current value
 		index  []int
 		canSet bool
 	}
@@ -362,6 +363,7 @@ func TestCanSetField(t *testing.T) {
 			{[]int{0}, false},
 			{[]int{0, 0}, false},
 			{[]int{0, 1}, true},
+			{[]int{0, -1, 1}, true},
 			{[]int{1}, false},
 			{[]int{2}, true},
 		},
@@ -371,6 +373,7 @@ func TestCanSetField(t *testing.T) {
 			{[]int{0}, false},
 			{[]int{0, 0}, false},
 			{[]int{0, 1}, true},
+			{[]int{0, -1, 1}, true},
 			{[]int{1}, false},
 			{[]int{2}, true},
 		},
@@ -380,6 +383,7 @@ func TestCanSetField(t *testing.T) {
 			{[]int{0}, true},
 			{[]int{0, 0}, false},
 			{[]int{0, 1}, true},
+			{[]int{0, -1, 1}, true},
 			{[]int{1}, false},
 			{[]int{2}, true},
 		},
@@ -402,7 +406,11 @@ func TestCanSetField(t *testing.T) {
 					if f.Kind() == Ptr {
 						f = f.Elem()
 					}
-					f = f.Field(i)
+					if i == -1 {
+						f = f.Addr().Elem()
+					} else {
+						f = f.Field(i)
+					}
 				}
 				if got := f.CanSet(); got != tc.canSet {
 					t.Errorf("CanSet() = %v, want %v", got, tc.canSet)