Hi ๐. If you're reading this, you may be looking for The GitHub Vet Project. This repository is only for testing purposes. None of this data is official.
There are false-negatives in this dataset, due to bugs fixed in the following commits/PRs:
Testing github-vet bots via issues
Hi ๐. If you're reading this, you may be looking for The GitHub Vet Project. This repository is only for testing purposes. None of this data is official.
There are false-negatives in this dataset, due to bugs fixed in the following commits/PRs:
Found a possible issue in G-Research/armada at internal/armada/scheduling/node_matching.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call at line 93 may store a reference to taint
Click here to see the code in its original context.
for _, taint := range taints {
// check only hard constraints
if taint.Effect == v1.TaintEffectPreferNoSchedule {
continue
}
if !tolerationsTolerateTaint(job.PodSpec.Tolerations, &taint) {
return false
}
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 46894df8cc09877d0166d4f393b3d6204c055512
Found a possible issue in SteveHuang27/ucore-x64-smp at go/src/cmd/gofix/reflect.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to x at line 181 may start a goroutine
Click here to see the code in its original context.
for i, x := range *n {
// Tentatively append to v; if we rewrite x
// we'll have to update the entry, so remember
// the index.
j := len(v)
v = append(v, x)
if reflectFixTypecheck(&x, insert, (*n)[i+1:]) {
// reflectFixTypecheck may have overwritten x.
// Update the entry we appended just before the call.
v[j] = x
fixed = true
}
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 5efdfb37d93cf6b746994cc7bc5700d964240951
Found a possible issue in box/error-reporting-with-kubernetes-events at vendor/golang.org/x/net/dns/dnsmessage/message_test.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to want at line 246 may start a goroutine
Click here to see the code in its original context.
for i, want := range wants {
b, err := want.Pack()
if err != nil {
t.Fatalf("%d: packing failed: %v", i, err)
}
var got Message
err = got.Unpack(b)
if err != nil {
t.Fatalf("%d: unpacking failed: %v", i, err)
}
if !reflect.DeepEqual(got, want) {
t.Errorf("%d: got = %+v, want = %+v", i, &got, &want)
}
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 8ae23931850d5b2ee820bd407afaac36826d5c75
Found a possible issue in openshift/ocp-release-operator-sdk at vendor/github.com/go-openapi/validate/example_validator.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to r at line 120 may start a goroutine
Click here to see the code in its original context.
for code, r := range op.Responses.StatusCodeResponses {
res.Merge(ex.validateExampleInResponse(&r, "response", path, code, op.ID))
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 91163b54561e4e434f66db4a34c464eabc06d2bd
Found a possible issue in jmoiron/euler.go at 013.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to num at line 129 may start a goroutine
Click here to see the code in its original context.
for _, num := range nums {
sum.Add(sum, &num)
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 70bc80e5e9f5d3c335d3e96e85c9b426cdda8d64
Found a possible issue in knative-sandbox/eventing-github at vendor/k8s.io/kube-openapi/pkg/generators/union.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to m at line 129 may start a goroutine
Click here to see the code in its original context.
for _, m := range t.Members {
if hasOpenAPITagValue(m.CommentLines, tagValueFalse) {
continue
}
if !shouldInlineMembers(&m) {
continue
}
u, err := parseUnions(m.Type)
unions = append(unions, u...)
errors = append(errors, err...)
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 8d7341cd0cdd0263348d8fce3e39c2ce3ca12c35
Found a possible issue in xbapps/xbvr at pkg/api/scenes.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to v at line 406 may start a goroutine
Click here to see the code in its original context.
for _, v := range scene.Tags {
db.Model(&scene).Association("Tags").Delete(&v)
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 34e872d8cfdc55f7ec4c930512da8e674b6e8682
Found a possible issue in knative-sandbox/eventing-github at vendor/k8s.io/kube-openapi/pkg/generators/openapi.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to m at line 289 may start a goroutine
Click here to see the code in its original context.
for _, m := range t.Members {
if hasOpenAPITagValue(m.CommentLines, tagValueFalse) {
continue
}
if shouldInlineMembers(&m) {
required, err = g.generateMembers(m.Type, required)
if err != nil {
return required, err
}
continue
}
name := getReferableName(&m)
if name == "" {
continue
}
if !hasOptionalTag(&m) {
required = append(required, name)
}
if err = g.generateProperty(&m, t); err != nil {
klog.Errorf("Error when generating: %v, %v\n", name, m)
return required, err
}
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 8d7341cd0cdd0263348d8fce3e39c2ce3ca12c35
Found a possible issue in SteveHuang27/ucore-x64-smp at go/src/pkg/index/suffixarray/suffixarray_test.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to tc at line 223 may start a goroutine
Click here to see the code in its original context.
for _, tc := range testCases {
x := New([]byte(tc.source))
testConstruction(t, &tc, x)
testLookups(t, &tc, x, 0)
testLookups(t, &tc, x, 1)
testLookups(t, &tc, x, 10)
testLookups(t, &tc, x, 2e9)
testLookups(t, &tc, x, -1)
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 5efdfb37d93cf6b746994cc7bc5700d964240951
Found a possible issue in SteveHuang27/ucore-x64-smp at go/src/pkg/regexp/all_test.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to test at line 93 may start a goroutine
Click here to see the code in its original context.
for _, test := range findTests {
matchTest(t, &test)
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 5efdfb37d93cf6b746994cc7bc5700d964240951
Found a possible issue in gutie1993/etherchain at vendor/github.com/gizak/termui/render.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to bucket at line 116 may start a goroutine
Click here to see the code in its original context.
for _, bucket := range buckets {
io.WriteString(os.Stdout, p.BucketHeader(&bucket, false, len(buckets) > 1))
io.WriteString(os.Stdout, p.StackLines(&bucket.Signature, srcLen, pkgLen, false))
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 674a86b846ba22a1faf180b94438b88184017b32
Found a possible issue in lightninglabs/chanleakcheck at vendor/github.com/lightninglabs/neutrino/blockntfns/manager.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
range-loop variable subscriber used in defer or goroutine at line 144
Click here to see the code in its original context.
for _, subscriber := range m.subscribers {
go func() {
defer wg.Done()
subscriber.cancel()
}()
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: d11d04ec57e60448848f2067413b03ceb0976b08
Found a possible issue in gutie1993/etherchain at eth/filters/api.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to log at line 254 may start a goroutine
Click here to see the code in its original context.
for _, log := range logs {
notifier.Notify(rpcSub.ID, &log)
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 674a86b846ba22a1faf180b94438b88184017b32
Found a possible issue in box/error-reporting-with-kubernetes-events at vendor/golang.org/x/net/lif/link_test.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to ll at line 51 may start a goroutine
Click here to see the code in its original context.
for _, ll := range lls {
if ll.Name != sll.Name || ll.Index != sll.Index {
t.Errorf("af=%s got %v; want %v", addrFamily(lp.af), &ll, &sll)
continue
}
t.Logf("af=%s name=%s %v", addrFamily(lp.af), sll.Name, &ll)
n++
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 8ae23931850d5b2ee820bd407afaac36826d5c75
Found a possible issue in pandabox-dev/SODA at SODA_code/go-ethereum/eth/filters/api.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to log at line 259 may start a goroutine
Click here to see the code in its original context.
for _, log := range logs {
notifier.Notify(rpcSub.ID, &log)
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 4be31c1ab2692500906684f3e8e3c502286bd1f5
Found a possible issue in openshift/ocp-release-operator-sdk at vendor/k8s.io/apiextensions-apiserver/pkg/apis/apiextensions/validation/validation.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call at line 1161 may store a reference to s
Click here to see the code in its original context.
for _, s := range s.Items.JSONSchemas {
if schemaHas(&s, pred) {
return true
}
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 91163b54561e4e434f66db4a34c464eabc06d2bd
Found a possible issue in openshift/ocp-release-operator-sdk at vendor/helm.sh/helm/v3/internal/experimental/registry/cache.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
reference to desc is reassigned at line 111
Click here to see the code in its original context.
for _, desc := range cache.ociStore.ListReferences() {
if desc.Annotations[ocispec.AnnotationRefName] == r.Name {
r.Exists = true
manifestBytes, err := cache.fetchBlob(&desc)
if err != nil {
return &r, err
}
var manifest ocispec.Manifest
err = json.Unmarshal(manifestBytes, &manifest)
if err != nil {
return &r, err
}
r.Manifest = &desc
r.Config = &manifest.Config
numLayers := len(manifest.Layers)
if numLayers != 1 {
return &r, errors.New(
fmt.Sprintf("manifest does not contain exactly 1 layer (total: %d)", numLayers))
}
var contentLayer *ocispec.Descriptor
for _, layer := range manifest.Layers {
switch layer.MediaType {
case HelmChartContentLayerMediaType:
contentLayer = &layer
}
}
if contentLayer == nil {
return &r, errors.New(
fmt.Sprintf("manifest does not contain a layer with mediatype %s", HelmChartContentLayerMediaType))
}
if contentLayer.Size == 0 {
return &r, errors.New(
fmt.Sprintf("manifest layer with mediatype %s is of size 0", HelmChartContentLayerMediaType))
}
r.ContentLayer = contentLayer
info, err := cache.ociStore.Info(ctx(cache.out, cache.debug), contentLayer.Digest)
if err != nil {
return &r, err
}
r.Size = info.Size
r.Digest = info.Digest
r.CreatedAt = info.CreatedAt
contentBytes, err := cache.fetchBlob(contentLayer)
if err != nil {
return &r, err
}
ch, err := loader.LoadArchive(bytes.NewBuffer(contentBytes))
if err != nil {
return &r, err
}
r.Chart = ch
}
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 91163b54561e4e434f66db4a34c464eabc06d2bd
Found a possible issue in SteveHuang27/ucore-x64-smp at go/src/pkg/time/time_test.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to test at line 278 may start a goroutine
Click here to see the code in its original context.
for _, test := range parseTests {
time, err := Parse(test.format, test.value)
if err != nil {
t.Errorf("%s error: %v", test.name, err)
} else {
checkTime(time, &test, t)
}
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 5efdfb37d93cf6b746994cc7bc5700d964240951
Found a possible issue in SteveHuang27/ucore-x64-smp at go/test/escape.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
reference to v is reassigned at line 128
Click here to see the code in its original context.
for k, v := range a {
p[k] = &v
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 5efdfb37d93cf6b746994cc7bc5700d964240951
Found a possible issue in nikhita/custom-database-controller at vendor/k8s.io/apimachinery/pkg/util/net/interface.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to intf at line 258 may start a goroutine
Click here to see the code in its original context.
for _, intf := range intfs {
if !isInterfaceUp(&intf) {
glog.V(4).Infof("Skipping: down interface %q", intf.Name)
continue
}
if isLoopbackOrPointToPoint(&intf) {
glog.V(4).Infof("Skipping: LB or P2P interface %q", intf.Name)
continue
}
addrs, err := nw.Addrs(&intf)
if err != nil {
return nil, err
}
if len(addrs) == 0 {
glog.V(4).Infof("Skipping: no addresses on interface %q", intf.Name)
continue
}
for _, addr := range addrs {
ip, _, err := net.ParseCIDR(addr.String())
if err != nil {
return nil, fmt.Errorf("Unable to parse CIDR for interface %q: %s", intf.Name, err)
}
if !memberOf(ip, family) {
glog.V(4).Infof("Skipping: no address family match for %q on interface %q.", ip, intf.Name)
continue
}
// TODO: Decide if should open up to allow IPv6 LLAs in future.
if !ip.IsGlobalUnicast() {
glog.V(4).Infof("Skipping: non-global address %q on interface %q.", ip, intf.Name)
continue
}
glog.V(4).Infof("Found global unicast address %q on interface %q.", ip, intf.Name)
return ip, nil
}
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 6f25e67af8f223c30eed34e5d21d13ec73578373
Found a possible issue in wheelcomplex/fswatcher at fswatcher.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
range-loop variable lazyPath used in defer or goroutine at line 667
Click here to see the code in its original context.
for lazyPath, _ := range lazyList {
delete(lazyList, lazyPath)
lazygw.Add(1)
subscan++
go func() {
//fmt.Printf("watchMgr, new lazyWatch %v || %v\n", lazyPath, rescann)
//self.Dump("befor lazyRWatch" + lazyPath)
if nw := self.lazyRWatch(lazyPath); nw > 0 {
fmt.Printf("watchMgr, new lazyWatch %v || %v\n", nw, lazyPath)
//self.Dump("after lazyRWatch" + lazyPath)
}
//fmt.Printf("watchMgr, end lazyWatch %v || %v\n", lazyPath, rescann)
lazygw.Done()
}()
//fmt.Printf("watchMgr, new lazyWatch %v end.\n", lazyPath)
if self.lazyAddStack.GetCacheSize() >= 2048 {
println("lazyRWatch abort for too many new events", self.lazyAddStack.GetCacheSize())
break
}
if subscan > 2 {
subscan = 0
// blocking
lazygw.Wait()
}
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: fc15843ba6dfd2d1aefb9595f157dee32f708234
Found a possible issue in SteveHuang27/ucore-x64-smp at go/src/pkg/sync/atomic/atomic_test.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
range-loop variable tt used in defer or goroutine at line 455
Click here to see the code in its original context.
for _, tt := range hammer32 {
if tt.f == nil {
continue
}
c := make(chan int)
var val uint32
for i := 0; i < p; i++ {
go func() {
tt.f(&val, n)
c <- 1
}()
}
for i := 0; i < p; i++ {
<-c
}
if val != uint32(n)*p {
t.Errorf("%s: val=%d want %d", tt.name, val, n*p)
}
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 5efdfb37d93cf6b746994cc7bc5700d964240951
Found a possible issue in openshift/ocp-release-operator-sdk at vendor/k8s.io/apimachinery/pkg/util/net/interface.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to intf at line 292 may start a goroutine
Click here to see the code in its original context.
for _, intf := range intfs {
if !isInterfaceUp(&intf) {
klog.V(4).Infof("Skipping: down interface %q", intf.Name)
continue
}
if isLoopbackOrPointToPoint(&intf) {
klog.V(4).Infof("Skipping: LB or P2P interface %q", intf.Name)
continue
}
addrs, err := nw.Addrs(&intf)
if err != nil {
return nil, err
}
if len(addrs) == 0 {
klog.V(4).Infof("Skipping: no addresses on interface %q", intf.Name)
continue
}
for _, addr := range addrs {
ip, _, err := net.ParseCIDR(addr.String())
if err != nil {
return nil, fmt.Errorf("Unable to parse CIDR for interface %q: %s", intf.Name, err)
}
if !memberOf(ip, family) {
klog.V(4).Infof("Skipping: no address family match for %q on interface %q.", ip, intf.Name)
continue
}
// TODO: Decide if should open up to allow IPv6 LLAs in future.
if !ip.IsGlobalUnicast() {
klog.V(4).Infof("Skipping: non-global address %q on interface %q.", ip, intf.Name)
continue
}
klog.V(4).Infof("Found global unicast address %q on interface %q.", ip, intf.Name)
return ip, nil
}
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 91163b54561e4e434f66db4a34c464eabc06d2bd
Found a possible issue in openshift/hypershift-toolkit at vendor/github.com/openshift/oc/pkg/cli/image/manifest/manifest.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to manifest at line 340 may start a goroutine
Click here to see the code in its original context.
for _, manifest := range t.Manifests {
if !filterFn(&manifest, len(t.Manifests) > 1) {
klog.V(5).Infof("Skipping image for %#v from %s", manifest.Platform, ref)
continue
}
klog.V(5).Infof("Including image for %#v from %s", manifest.Platform, ref)
filtered = append(filtered, manifest)
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 148be6f31a365dbcdf1cbd647773f59ccbcc282a
Found a possible issue in lxfontes/poconetes at controllers/application_controller.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to svc at line 118 may start a goroutine
Click here to see the code in its original context.
for _, svc := range serviceList.Items {
if !metav1.IsControlledBy(&svc, app) {
continue
}
formation, ok := svc.GetLabels()[appsv1.LabelFormation]
if !ok {
continue
}
var found bool
for _, f := range app.Spec.Formations {
if f.Name == formation {
found = true
break
}
}
if !found {
deleteService = append(deleteService, svc)
}
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: b27e348daa6908d01b93643b9a8efa6e128556a1
Found a possible issue in knative-sandbox/eventing-github at vendor/knative.dev/eventing/test/lib/test_runner.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to e at line 215 may start a goroutine
Click here to see the code in its original context.
for _, e := range items {
client.T.Log(formatEvent(&e))
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 8d7341cd0cdd0263348d8fce3e39c2ce3ca12c35
Found a possible issue in openshift/hypershift-toolkit at vendor/k8s.io/apimachinery/pkg/util/net/interface.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to intf at line 280 may start a goroutine
Click here to see the code in its original context.
for _, intf := range intfs {
if !isInterfaceUp(&intf) {
klog.V(4).Infof("Skipping: down interface %q", intf.Name)
continue
}
if isLoopbackOrPointToPoint(&intf) {
klog.V(4).Infof("Skipping: LB or P2P interface %q", intf.Name)
continue
}
addrs, err := nw.Addrs(&intf)
if err != nil {
return nil, err
}
if len(addrs) == 0 {
klog.V(4).Infof("Skipping: no addresses on interface %q", intf.Name)
continue
}
for _, addr := range addrs {
ip, _, err := net.ParseCIDR(addr.String())
if err != nil {
return nil, fmt.Errorf("Unable to parse CIDR for interface %q: %s", intf.Name, err)
}
if !memberOf(ip, family) {
klog.V(4).Infof("Skipping: no address family match for %q on interface %q.", ip, intf.Name)
continue
}
// TODO: Decide if should open up to allow IPv6 LLAs in future.
if !ip.IsGlobalUnicast() {
klog.V(4).Infof("Skipping: non-global address %q on interface %q.", ip, intf.Name)
continue
}
klog.V(4).Infof("Found global unicast address %q on interface %q.", ip, intf.Name)
return ip, nil
}
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 148be6f31a365dbcdf1cbd647773f59ccbcc282a
Found a possible issue in SteveHuang27/ucore-x64-smp at go/src/pkg/regexp/find_test.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to test at line 185 may start a goroutine
Click here to see the code in its original context.
for _, test := range findTests {
testFindIndex(&test, MustCompile(test.pat).FindIndex([]byte(test.text)), t)
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 5efdfb37d93cf6b746994cc7bc5700d964240951
Found a possible issue in argoproj-labs/applicationset at pkg/controllers/applicationset_controller_test.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to a at line 332 may start a goroutine
Click here to see the code in its original context.
for _, a := range c.existsApps {
controllerutil.SetControllerReference(&c.appSet, &a, scheme)
initObjs = append(initObjs, &a)
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 57b55801c884341b60053f69f440ab795f94047b
Found a possible issue in openshift/ocp-release-operator-sdk at vendor/github.com/go-openapi/analysis/analyzer.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to pathItem at line 215 may start a goroutine
Click here to see the code in its original context.
for path, pathItem := range s.AllPaths() {
s.analyzeOperations(path, &pathItem)
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 91163b54561e4e434f66db4a34c464eabc06d2bd
Found a possible issue in openshift/ocp-release-operator-sdk at vendor/helm.sh/helm/v3/pkg/kube/wait.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to pod at line 158 may start a goroutine
Click here to see the code in its original context.
for _, pod := range pods {
if !w.isPodReady(&pod) {
return false, nil
}
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 91163b54561e4e434f66db4a34c464eabc06d2bd
Found a possible issue in G-Research/armada at internal/executor/fake/context/context.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call at line 304 may store a reference to t
Click here to see the code in its original context.
for _, t := range n.Spec.Taints {
// check only hard constraints
if t.Effect == v1.TaintEffectPreferNoSchedule {
continue
}
if !tolerationsTolerateTaint(pod.Spec.Tolerations, &t) {
return false
}
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 46894df8cc09877d0166d4f393b3d6204c055512
Found a possible issue in lvtanxi/ThreePartyLibraries at gorm_example/crud.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to user at line 18 may start a goroutine
Click here to see the code in its original context.
for _, user := range users {
db.Create(&user)
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 208d2f01262071733baf3e571d988e9f6365925b
Found a possible issue in SteveHuang27/ucore-x64-smp at go/src/pkg/exp/regexp/find_test.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to test at line 185 may start a goroutine
Click here to see the code in its original context.
for _, test := range findTests {
testFindIndex(&test, MustCompile(test.pat).FindIndex([]byte(test.text)), t)
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 5efdfb37d93cf6b746994cc7bc5700d964240951
Found a possible issue in DevopsArtFactory/klocust at vendor/k8s.io/apimachinery/pkg/util/net/interface.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to intf at line 292 may start a goroutine
Click here to see the code in its original context.
for _, intf := range intfs {
if !isInterfaceUp(&intf) {
klog.V(4).Infof("Skipping: down interface %q", intf.Name)
continue
}
if isLoopbackOrPointToPoint(&intf) {
klog.V(4).Infof("Skipping: LB or P2P interface %q", intf.Name)
continue
}
addrs, err := nw.Addrs(&intf)
if err != nil {
return nil, err
}
if len(addrs) == 0 {
klog.V(4).Infof("Skipping: no addresses on interface %q", intf.Name)
continue
}
for _, addr := range addrs {
ip, _, err := net.ParseCIDR(addr.String())
if err != nil {
return nil, fmt.Errorf("Unable to parse CIDR for interface %q: %s", intf.Name, err)
}
if !memberOf(ip, family) {
klog.V(4).Infof("Skipping: no address family match for %q on interface %q.", ip, intf.Name)
continue
}
// TODO: Decide if should open up to allow IPv6 LLAs in future.
if !ip.IsGlobalUnicast() {
klog.V(4).Infof("Skipping: non-global address %q on interface %q.", ip, intf.Name)
continue
}
klog.V(4).Infof("Found global unicast address %q on interface %q.", ip, intf.Name)
return ip, nil
}
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: becb14d273085674823c3e2e418fcbd82034bfb6
Found a possible issue in knative-sandbox/eventing-github at vendor/github.com/prometheus/statsd_exporter/pkg/mapper/mapper.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to mapping at line 371 may start a goroutine
Click here to see the code in its original context.
for _, mapping := range m.Mappings {
// if a rule don't have regex matching type, the regex field is unset
if mapping.regex == nil {
continue
}
matches := mapping.regex.FindStringSubmatchIndex(statsdMetric)
if len(matches) == 0 {
continue
}
mapping.Name = string(mapping.regex.ExpandString(
[]byte{},
mapping.Name,
statsdMetric,
matches,
))
if mt := mapping.MatchMetricType; mt != "" && mt != statsdMetricType {
continue
}
labels := prometheus.Labels{}
for label, valueExpr := range mapping.Labels {
value := mapping.regex.ExpandString([]byte{}, valueExpr, statsdMetric, matches)
labels[label] = string(value)
}
m.cache.AddMatch(statsdMetric, statsdMetricType, &mapping, labels)
return &mapping, labels, true
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 8d7341cd0cdd0263348d8fce3e39c2ce3ca12c35
Found a possible issue in pandabox-dev/SODA at SODA_code/go-ethereum/vendor/github.com/go-ole/go-ole/safearrayslices.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to v at line 17 may start a goroutine
Click here to see the code in its original context.
for i, v := range slice {
safeArrayPutElement(array, int64(i), uintptr(unsafe.Pointer(&v)))
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 4be31c1ab2692500906684f3e8e3c502286bd1f5
Found a possible issue in box/error-reporting-with-kubernetes-events at vendor/k8s.io/kube-openapi/pkg/generators/openapi.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to m at line 331 may start a goroutine
Click here to see the code in its original context.
for _, m := range t.Members {
if hasOpenAPITagValue(m.CommentLines, tagValueFalse) {
continue
}
if shouldInlineMembers(&m) {
required, err = g.generateMembers(m.Type, required)
if err != nil {
return required, err
}
continue
}
name := getReferableName(&m)
if name == "" {
continue
}
if !hasOptionalTag(&m) {
required = append(required, name)
}
if err = g.generateProperty(&m, t); err != nil {
glog.Errorf("Error when generating: %v, %v\n", name, m)
return required, err
}
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 8ae23931850d5b2ee820bd407afaac36826d5c75
Found a possible issue in trustedanalytics/application-broker at cloud/cloud-api.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to clone at line 117 may start a goroutine
Click here to see the code in its original context.
for clone := range results {
transaction.AddComponentClone(&clone)
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: e358542420c085f894a19f5a44217e087fabda37
Found a possible issue in PiterPentester/kubebuilderCronJob at controllers/cronjob_controller.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to job at line 129 may start a goroutine
Click here to see the code in its original context.
for i, job := range childJobs.Items {
_, finishedType := isJobFinished(&job)
switch finishedType {
case "": // ongoing
activeJobs = append(activeJobs, &childJobs.Items[i])
case kbatch.JobFailed:
failedJobs = append(failedJobs, &childJobs.Items[i])
case kbatch.JobComplete:
successfulJobs = append(successfulJobs, &childJobs.Items[i])
}
// We'll store the launch time in an annotation, so we'll reconstitute that from
// the active jobs themselves.
scheduledTimeForJob, err := getScheduledTimeForJob(&job)
if err != nil {
log.Error(err, "unable to parse schedule time for child job", "job", &job)
continue
}
if scheduledTimeForJob != nil {
if mostRecentTime == nil {
mostRecentTime = scheduledTimeForJob
} else if mostRecentTime.Before(*scheduledTimeForJob) {
mostRecentTime = scheduledTimeForJob
}
}
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 2f475ecd9db6fe7e123b9b05f03233098a46d448
Found a possible issue in travis-ci/2fabot at vendor/github.com/lsegal/gucumber/runner.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
range-loop variable step used in defer or goroutine at line 310
Click here to see the code in its original context.
for _, step := range s.Steps {
errCount := len(t.errors)
found := false
if !skipping && !t.Failed() {
done := make(chan bool)
go func() {
defer func() {
c.Results = append(c.Results, RunnerResult{t, f, s})
if t.Skipped() {
c.SkipCount++
skipping = true
clr = clrYellow
} else if t.Failed() {
c.FailCount++
clr = clrRed
}
done <- true
}()
f, err := c.Execute(t, step.Text, string(step.Argument))
if err != nil {
t.Error(err)
}
found = f
if !f {
t.Skip("no match function for step")
}
}()
<-done
}
if skipping && !found {
cstep := step
c.Unmatched = append(c.Unmatched, &cstep)
}
if !isExample {
c.fileLine(clr, " %s %s", step.Filename, step.Line,
s.LongestLine(), step.Type, step.Text)
if len(step.Argument) > 0 {
if !step.Argument.IsTabular() {
c.line(clrWhite, ` """`)
}
for _, l := range strings.Split(string(step.Argument), "\n") {
c.line(clrWhite, " %s", l)
}
if !step.Argument.IsTabular() {
c.line(clrWhite, ` """`)
}
}
}
if len(t.errors) > errCount {
c.line(clrRed, "\n"+t.errors[len(t.errors)-1].message)
}
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 2949646ec078a4b157e205b6048f2c80aabe9d6c
Found a possible issue in torusresearch/torus-node at pvss/pvss.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to scalar at line 497 may start a goroutine
Click here to see the code in its original context.
for _, scalar := range scalars {
sumScalar.Add(sumScalar, &scalar)
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: d9fe26e7d8714fe199a893cd7667c0f46a775620
Found a possible issue in SteveHuang27/ucore-x64-smp at go/src/pkg/archive/zip/reader_test.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
range-loop variable ft used in defer or goroutine at line 125
Click here to see the code in its original context.
for j, ft := range zt.File {
go func() {
readTestFile(t, ft, z.File[j])
done <- true
}()
n++
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 5efdfb37d93cf6b746994cc7bc5700d964240951
Found a possible issue in openshift/ocp-release-operator-sdk at vendor/github.com/go-openapi/validate/default_validator.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to r at line 144 may start a goroutine
Click here to see the code in its original context.
for code, r := range op.Responses.StatusCodeResponses {
res.Merge(d.validateDefaultInResponse(&r, "response", path, code, op.ID))
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 91163b54561e4e434f66db4a34c464eabc06d2bd
Found a possible issue in SteveHuang27/ucore-x64-smp at go/src/pkg/exp/regexp/all_test.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to test at line 96 may start a goroutine
Click here to see the code in its original context.
for _, test := range findTests {
matchTest(t, &test)
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 5efdfb37d93cf6b746994cc7bc5700d964240951
Found a possible issue in gutie1993/etherchain at vendor/github.com/naoina/go-stringutil/da.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
reference to r is reassigned at line 200
Click here to see the code in its original context.
for i, r := range records {
if len(r.key) <= depth {
leaf = &r
continue
}
c := r.key[depth]
switch {
case pc < c:
sib = append(sib, sibling{start: i, c: c})
case pc == c:
continue
default:
return nil, nil, fmt.Errorf("stringutil: BUG: records hasn't been sorted")
}
if n > 0 {
sib[n-1].end = i
}
pc = c
n++
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 674a86b846ba22a1faf180b94438b88184017b32
Found a possible issue in box/error-reporting-with-kubernetes-events at vendor/k8s.io/client-go/rest/watch/decoder_test.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
range-loop variable eventType used in defer or goroutine at line 61
Click here to see the code in its original context.
for _, eventType := range table {
out, in := io.Pipe()
decoder := restclientwatch.NewDecoder(streaming.NewDecoder(out, getDecoder()), getDecoder())
expect := &v1.Pod{ObjectMeta: metav1.ObjectMeta{Name: "foo"}}
encoder := json.NewEncoder(in)
go func() {
data, err := runtime.Encode(scheme.Codecs.LegacyCodec(v1.SchemeGroupVersion), expect)
if err != nil {
t.Fatalf("Unexpected error %v", err)
}
event := metav1.WatchEvent{
Type: string(eventType),
Object: runtime.RawExtension{Raw: json.RawMessage(data)},
}
if err := encoder.Encode(&event); err != nil {
t.Errorf("Unexpected error %v", err)
}
in.Close()
}()
done := make(chan struct{})
go func() {
action, got, err := decoder.Decode()
if err != nil {
t.Fatalf("Unexpected error %v", err)
}
if e, a := eventType, action; e != a {
t.Errorf("Expected %v, got %v", e, a)
}
if e, a := expect, got; !apiequality.Semantic.DeepDerivative(e, a) {
t.Errorf("Expected %v, got %v", e, a)
}
t.Logf("Exited read")
close(done)
}()
<-done
done = make(chan struct{})
go func() {
_, _, err := decoder.Decode()
if err == nil {
t.Errorf("Unexpected nil error")
}
close(done)
}()
<-done
decoder.Close()
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 8ae23931850d5b2ee820bd407afaac36826d5c75
Found a possible issue in Toasterson/pkg6-go at depotd/daemon.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
reference to mirror is reassigned at line 67
Click here to see the code in its original context.
for name, mirror := range d.State.Mirrors {
d.Mirrors[name] = &mirror
d.Mirrors[name].InitRepos()
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 34010959150f1025ebdf6348539f6f72e5ecd33e
Found a possible issue in zahidaliayub/skycoin at src/visor/historydb/transaction_test.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to tx at line 88 may start a goroutine
Click here to see the code in its original context.
for _, tx := range txs[:2] {
require.Nil(t, txsBkt.Add(&tx))
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 4a11cd3a2841bbb555734c6889838fefabfa1b41
Found a possible issue in box/error-reporting-with-kubernetes-events at vendor/golang.org/x/net/icmp/multipart_test.go
Below is the message reported by the analyzer for this snippet of code. Beware that the analyzer only reports the first
issue it finds, so please do not limit your consideration to the contents of the below message.
function call which takes a reference to tt at line 145 may start a goroutine
Click here to see the code in its original context.
for i, tt := range marshalAndParseMultipartMessageForIPv4Tests {
b, err := tt.Marshal(nil)
if err != nil {
t.Fatal(err)
}
if b[5] != 32 {
t.Errorf("#%v: got %v; want 32", i, b[5])
}
m, err := icmp.ParseMessage(iana.ProtocolICMP, b)
if err != nil {
t.Fatal(err)
}
if m.Type != tt.Type || m.Code != tt.Code {
t.Errorf("#%v: got %v; want %v", i, m, &tt)
}
switch m.Type {
case ipv4.ICMPTypeDestinationUnreachable:
got, want := m.Body.(*icmp.DstUnreach), tt.Body.(*icmp.DstUnreach)
if !reflect.DeepEqual(got.Extensions, want.Extensions) {
t.Error(dumpExtensions(i, got.Extensions, want.Extensions))
}
if len(got.Data) != 128 {
t.Errorf("#%v: got %v; want 128", i, len(got.Data))
}
case ipv4.ICMPTypeTimeExceeded:
got, want := m.Body.(*icmp.TimeExceeded), tt.Body.(*icmp.TimeExceeded)
if !reflect.DeepEqual(got.Extensions, want.Extensions) {
t.Error(dumpExtensions(i, got.Extensions, want.Extensions))
}
if len(got.Data) != 128 {
t.Errorf("#%v: got %v; want 128", i, len(got.Data))
}
case ipv4.ICMPTypeParameterProblem:
got, want := m.Body.(*icmp.ParamProb), tt.Body.(*icmp.ParamProb)
if !reflect.DeepEqual(got.Extensions, want.Extensions) {
t.Error(dumpExtensions(i, got.Extensions, want.Extensions))
}
if len(got.Data) != 128 {
t.Errorf("#%v: got %v; want 128", i, len(got.Data))
}
}
}
Leave a reaction on this issue to contribute to the project by classifying this instance as a Bug ๐, Mitigated ๐, or Desirable Behavior ๐
See the descriptions of the classifications here for more information.
commit ID: 8ae23931850d5b2ee820bd407afaac36826d5c75
A declarative, efficient, and flexible JavaScript library for building user interfaces.
๐ Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
An Open Source Machine Learning Framework for Everyone
The Web framework for perfectionists with deadlines.
A PHP framework for web artisans
Bring data to life with SVG, Canvas and HTML. ๐๐๐
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
Some thing interesting about web. New door for the world.
A server is a program made to process requests and deliver data to clients.
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
Some thing interesting about visualization, use data art
Some thing interesting about game, make everyone happy.
We are working to build community through open source technology. NB: members must have two-factor auth.
Open source projects and samples from Microsoft.
Google โค๏ธ Open Source for everyone.
Alibaba Open Source for everyone
Data-Driven Documents codes.
China tencent open source team.