service/dap: do not try to access value of unreadable string variables

aarzilli · aarzilli · commit 4d6631616854 · 2025-07-14T11:43:15.000+02:00
Fixes #4053
diff --git a/service/dap/server.go b/service/dap/server.go
@@ -2437,8 +2437,10 @@ func (s *Session) childrenToDAPVariables(v *fullyQualifiedVariable) []dap.Variab
 				reflect.Uint, reflect.Uint8, reflect.Uint16, reflect.Uint32, reflect.Uint64, reflect.Uintptr:
 				valexpr = fmt.Sprintf("%s[%s]", v.fullyQualifiedNameOrExpr, api.VariableValueAsString(keyv))
 			case reflect.String:
-				if key := constant.StringVal(keyv.Value); keyv.Len == int64(len(key)) { // fully loaded
-					valexpr = fmt.Sprintf("%s[%q]", v.fullyQualifiedNameOrExpr, key)
+				if keyv.Unreadabel == nil {
+					if key := constant.StringVal(keyv.Value); keyv.Len == int64(len(key)) { // fully loaded
+						valexpr = fmt.Sprintf("%s[%q]", v.fullyQualifiedNameOrExpr, key)
+					}
 				}
 			}
 			key, keyref := s.convertVariable(keyv, keyexpr)

Original file line number	Diff line number	Diff line change
`@@ -2437,8 +2437,10 @@ func (s Session) childrenToDAPVariables(v fullyQualifiedVariable) []dap.Variab`
`2437`	`2437`	`reflect.Uint, reflect.Uint8, reflect.Uint16, reflect.Uint32, reflect.Uint64, reflect.Uintptr:`
`2438`	`2438`	`valexpr = fmt.Sprintf("%s[%s]", v.fullyQualifiedNameOrExpr, api.VariableValueAsString(keyv))`
`2439`	`2439`	`case reflect.String:`
`2440`		`- if key := constant.StringVal(keyv.Value); keyv.Len == int64(len(key)) { // fully loaded`
`2441`		`- valexpr = fmt.Sprintf("%s[%q]", v.fullyQualifiedNameOrExpr, key)`
	`2440`	`+ if keyv.Unreadabel == nil {`
	`2441`	`+ if key := constant.StringVal(keyv.Value); keyv.Len == int64(len(key)) { // fully loaded`
	`2442`	`+ valexpr = fmt.Sprintf("%s[%q]", v.fullyQualifiedNameOrExpr, key)`
	`2443`	`+ }`
`2442`	`2444`	`}`
`2443`	`2445`	`}`
`2444`	`2446`	`key, keyref := s.convertVariable(keyv, keyexpr)`