mirror of
https://github.com/tailscale/tailscale.git
synced 2024-11-26 03:25:35 +00:00
d32700c7b2
There are 5 types that we care about that implement AppendTo: key.DiscoPublic key.NodePublic netip.Prefix netipx.IPRange netip.Addr The key types are thin wrappers around [32]byte and are memory hashable. The netip.Prefix and netipx.IPRange types are thin wrappers over netip.Addr and are hashable by default if netip.Addr is hashable. The netip.Addr type is the only one with a complex structure where the default behavior of deephash does not hash it correctly due to the presence of the intern.Value type. Drop support for AppendTo and instead add specialized hashing for netip.Addr that would be semantically equivalent to == on the netip.Addr values. The AppendTo support was already broken prior to this change. It was fully removed (intentionally or not) in #4870. It was partially restored in #4858 for the fast path, but still broken in the slow path. Just drop support for it altogether. This does mean we lack any ability for types to self-hash themselves. In the future we can add support for types that implement: interface { DeepHash() Sum } Test and fuzz cases were added for the relevant types that used to rely on the AppendTo method. FuzzAddr has been executed on 1 billion samples without issues. Signed-off-by: Joe Tsai joetsai@digital-static.net
113 lines
3.0 KiB
Go
113 lines
3.0 KiB
Go
// Copyright (c) 2022 Tailscale Inc & AUTHORS All rights reserved.
|
|
// Use of this source code is governed by a BSD-style
|
|
// license that can be found in the LICENSE file.
|
|
|
|
package deephash
|
|
|
|
import (
|
|
"net/netip"
|
|
"reflect"
|
|
"time"
|
|
)
|
|
|
|
var (
|
|
timeTimeType = reflect.TypeOf((*time.Time)(nil)).Elem()
|
|
netipAddrType = reflect.TypeOf((*netip.Addr)(nil)).Elem()
|
|
)
|
|
|
|
// typeIsSpecialized reports whether this type has specialized hashing.
|
|
// These are never memory hashable and never considered recursive.
|
|
func typeIsSpecialized(t reflect.Type) bool {
|
|
switch t {
|
|
case timeTimeType, netipAddrType:
|
|
return true
|
|
default:
|
|
return false
|
|
}
|
|
}
|
|
|
|
// typeIsMemHashable reports whether t can be hashed by directly hashing its
|
|
// contiguous bytes in memory (e.g. structs with gaps are not mem-hashable).
|
|
func typeIsMemHashable(t reflect.Type) bool {
|
|
if typeIsSpecialized(t) {
|
|
return false
|
|
}
|
|
if t.Size() == 0 {
|
|
return true
|
|
}
|
|
switch t.Kind() {
|
|
case reflect.Bool,
|
|
reflect.Int, reflect.Int8, reflect.Int16, reflect.Int32, reflect.Int64,
|
|
reflect.Uint, reflect.Uint8, reflect.Uint16, reflect.Uint32, reflect.Uint64, reflect.Uintptr,
|
|
reflect.Float32, reflect.Float64,
|
|
reflect.Complex64, reflect.Complex128:
|
|
return true
|
|
case reflect.Array:
|
|
return typeIsMemHashable(t.Elem())
|
|
case reflect.Struct:
|
|
var sumFieldSize uintptr
|
|
for i, numField := 0, t.NumField(); i < numField; i++ {
|
|
sf := t.Field(i)
|
|
if !typeIsMemHashable(sf.Type) {
|
|
return false
|
|
}
|
|
sumFieldSize += sf.Type.Size()
|
|
}
|
|
return sumFieldSize == t.Size() // ensure no gaps
|
|
}
|
|
return false
|
|
}
|
|
|
|
// typeIsRecursive reports whether t has a path back to itself.
|
|
// For interfaces, it currently always reports true.
|
|
func typeIsRecursive(t reflect.Type) bool {
|
|
inStack := map[reflect.Type]bool{}
|
|
var visitType func(t reflect.Type) (isRecursiveSoFar bool)
|
|
visitType = func(t reflect.Type) (isRecursiveSoFar bool) {
|
|
// Check whether we have seen this type before.
|
|
if inStack[t] {
|
|
return true
|
|
}
|
|
inStack[t] = true
|
|
defer func() {
|
|
delete(inStack, t)
|
|
}()
|
|
|
|
// Types with specialized hashing are never considered recursive.
|
|
if typeIsSpecialized(t) {
|
|
return false
|
|
}
|
|
|
|
// Any type that is memory hashable must not be recursive since
|
|
// cycles can only occur if pointers are involved.
|
|
if typeIsMemHashable(t) {
|
|
return false
|
|
}
|
|
|
|
// Recursively check types that may contain pointers.
|
|
switch t.Kind() {
|
|
default:
|
|
panic("unhandled kind " + t.Kind().String())
|
|
case reflect.String, reflect.UnsafePointer, reflect.Func:
|
|
return false
|
|
case reflect.Interface:
|
|
// Assume the worst for now. TODO(bradfitz): in some cases
|
|
// we should be able to prove that it's not recursive. Not worth
|
|
// it for now.
|
|
return true
|
|
case reflect.Array, reflect.Chan, reflect.Pointer, reflect.Slice:
|
|
return visitType(t.Elem())
|
|
case reflect.Map:
|
|
return visitType(t.Key()) || visitType(t.Elem())
|
|
case reflect.Struct:
|
|
for i, numField := 0, t.NumField(); i < numField; i++ {
|
|
if visitType(t.Field(i).Type) {
|
|
return true
|
|
}
|
|
}
|
|
return false
|
|
}
|
|
}
|
|
return visitType(t)
|
|
}
|