tracer: Check for nil interface types in span.Finish(WithError())

ddtrace/tracer/option.go

@@ -15,6 +15,7 @@ import (
 	"net/url"
 	"os"
 	"path/filepath"
+	"reflect"
 	"runtime"
 	"strconv"
 	"strings"
@@ -939,8 +940,18 @@ func FinishTime(t time.Time) FinishOption {
 
 // WithError marks the span as having had an error. It uses the information from
 // err to set tags such as the error message, error type and stack trace. It has
-// no effect if the error is nil.
+// no effect if the error is nil or the underlying object is a nil pointer (https://go.dev/doc/faq#nil_error).
 func WithError(err error) FinishOption {
+	if err == nil {
+		return func(_ *ddtrace.FinishConfig) {}
+	}
+	v := reflect.ValueOf(err)
+	// Here be dragons: https://go.dev/doc/faq#nil_error
+	// https://github.com/DataDog/dd-trace-go/issues/2029
+	if !v.IsValid() || (v.Kind() == reflect.Ptr && v.IsNil()) {
+		log.Debug("Underlying error value is nil pointer, ignoring. See https://go.dev/doc/faq#nil_error for details.")
+		return func(_ *ddtrace.FinishConfig) {}
+	}
 	return func(cfg *ddtrace.FinishConfig) {
 		cfg.Error = err
 	}

ddtrace/tracer/span_test.go

@@ -230,6 +230,28 @@ func TestSpanFinishWithError(t *testing.T) {
 	assert.NotEmpty(span.Meta[ext.ErrorStack])
 }
 
+type MyError struct {
+	msg string
+}
+
+func (e *MyError) Error() string {
+	return e.msg
+}
+
+func TestSpanFinishWithErrorNilCustomError(t *testing.T) {
+	assert := assert.New(t)
+
+	var err error
+	var myError *MyError
+	myError = nil
+	err = myError
+
+	span := newBasicSpan("web.request")
+	span.Finish(WithError(err))
+
+	assert.Equal(int32(0), span.Error)
+}
+
 func TestSpanFinishWithErrorNoDebugStack(t *testing.T) {
 	assert := assert.New(t)