Move the second part of the nil check earlier in the function. This fixes a panic under certain conditions.

1. arg is a pointer to nil
2. The type of arg implements BinaryEncoder or TextEncoder
3. The implementation of that interface uses a non-pointer receiver

extended_query_builder.go

@@ -67,6 +67,13 @@ func (eqb *extendedQueryBuilder) encodeExtendedParamValue(ci *pgtype.ConnInfo, o
 		return nil, nil
 	}
 
+	refVal := reflect.ValueOf(arg)
+	argIsPtr := refVal.Kind() == reflect.Ptr
+
+	if argIsPtr && refVal.IsNil() {
+		return nil, nil
+	}
+
 	if eqb.paramValueBytes == nil {
 		eqb.paramValueBytes = make([]byte, 0, 128)
 	}
@@ -100,12 +107,9 @@ func (eqb *extendedQueryBuilder) encodeExtendedParamValue(ci *pgtype.ConnInfo, o
 		return []byte(arg), nil
 	}
 
-	refVal := reflect.ValueOf(arg)
+	if argIsPtr {
-
+		// We have already checked that arg is not pointing to nil,
-	if refVal.Kind() == reflect.Ptr {
+		// so it is safe to dereference here.
-		if refVal.IsNil() {
-			return nil, nil
-		}
 		arg = refVal.Elem().Interface()
 		return eqb.encodeExtendedParamValue(ci, oid, arg)
 	}