Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "v0.50.10" #59

Merged
merged 1 commit into from
Sep 24, 2024
Merged

Revert "v0.50.10" #59

merged 1 commit into from
Sep 24, 2024

Conversation

pr0n00gler
Copy link
Collaborator

Reverts #57

halt = true

case app.haltTime > 0 && time.Unix() >= int64(app.haltTime):
case app.haltTime > 0 && time.Unix() > int64(app.haltTime):

Check failure

Code scanning / Golang security checks by gosec

integer overflow conversion uint64 -> uint8 Error

integer overflow conversion uint64 -> int64
break
}
}
fieldNum := int32(wire >> 3)

Check failure

Code scanning / Golang security checks by gosec

integer overflow conversion uint64 -> uint8 Error test

integer overflow conversion uint64 -> int32
}
}
fieldNum := int32(wire >> 3)
wireType := int(wire & 0x7)

Check failure

Code scanning / Golang security checks by gosec

integer overflow conversion uint64 -> uint8 Error test

integer overflow conversion uint64 -> int
break
}
}
fieldNum := int32(wire >> 3)

Check failure

Code scanning / Golang security checks by gosec

integer overflow conversion uint64 -> uint8 Error test

integer overflow conversion uint64 -> int32
}
}
fieldNum := int32(wire >> 3)
wireType := int(wire & 0x7)

Check failure

Code scanning / Golang security checks by gosec

integer overflow conversion uint64 -> uint8 Error test

integer overflow conversion uint64 -> int
}
}
fieldNum := int32(wire >> 3)
wireType := int(wire & 0x7)

Check failure

Code scanning / Golang security checks by gosec

integer overflow conversion uint64 -> uint8 Error test

integer overflow conversion uint64 -> int
break
}
}
fieldNum := int32(wire >> 3)

Check failure

Code scanning / Golang security checks by gosec

integer overflow conversion uint64 -> uint8 Error test

integer overflow conversion uint64 -> int32
}
}
fieldNum := int32(wire >> 3)
wireType := int(wire & 0x7)

Check failure

Code scanning / Golang security checks by gosec

integer overflow conversion uint64 -> uint8 Error test

integer overflow conversion uint64 -> int
break
}
}
fieldNum := int32(wire >> 3)

Check failure

Code scanning / Golang security checks by gosec

integer overflow conversion uint64 -> uint8 Error test

integer overflow conversion uint64 -> int32
}
}
fieldNum := int32(wire >> 3)
wireType := int(wire & 0x7)

Check failure

Code scanning / Golang security checks by gosec

integer overflow conversion uint64 -> uint8 Error test

integer overflow conversion uint64 -> int
Comment on lines +119 to +121
for name := range modules {
mods = append(mods, " "+string(name))
}

Check warning

Code scanning / CodeQL

Iteration over map Warning

Iteration over map may be a possible source of non-determinism
@@ -1,7 +1,11 @@
package appmodule

import (
depinjectappconfig "cosmossdk.io/depinject/appconfig"
"reflect"

Check notice

Code scanning / CodeQL

Sensitive package import Note

Certain system packages contain functions which may be a possible source of non-determinism

import (
"fmt"
"reflect"

Check notice

Code scanning / CodeQL

Sensitive package import Note

Certain system packages contain functions which may be a possible source of non-determinism
Comment on lines +32 to +55
for _, initializer := range ModuleRegistry {
descriptor := initializer.ConfigProtoMessage.ProtoReflect().Descriptor()
fullName := descriptor.FullName()
if _, ok := res[fullName]; ok {
return nil, fmt.Errorf("duplicate module registratio for %s", fullName)
}

modDesc := proto.GetExtension(descriptor.Options(), appv1alpha1.E_Module).(*appv1alpha1.ModuleDescriptor)
if modDesc == nil {
return nil, fmt.Errorf(
"protobuf type %s registered as a module should have the option %s",
fullName,
appv1alpha1.E_Module.TypeDescriptor().FullName())
}

if modDesc.GoImport == "" {
return nil, fmt.Errorf(
"protobuf type %s registered as a module should have ModuleDescriptor.go_import specified",
fullName,
)
}

res[fullName] = initializer
}

Check warning

Code scanning / CodeQL

Iteration over map Warning

Iteration over map may be a possible source of non-determinism
protoiface "google.golang.org/protobuf/runtime/protoiface"
protoimpl "google.golang.org/protobuf/runtime/protoimpl"
io "io"
reflect "reflect"

Check notice

Code scanning / CodeQL

Sensitive package import Note test

Certain system packages contain functions which may be a possible source of non-determinism
if fd.IsExtension() {
panic(fmt.Errorf("proto3 declared messages do not support extensions: testpb.TestUnregisteredModule"))
}
panic(fmt.Errorf("message testpb.TestUnregisteredModule does not contain field %s", fd.FullName()))

Check warning

Code scanning / CodeQL

Panic in BeginBock or EndBlock consensus methods Warning test

Possible panics in BeginBock- or EndBlock-related consensus methods could cause a chain halt
switch fd.FullName() {
default:
if fd.IsExtension() {
panic(fmt.Errorf("proto3 declared messages do not support extensions: testpb.TestNoModuleOptionModule"))

Check warning

Code scanning / CodeQL

Panic in BeginBock or EndBlock consensus methods Warning test

Possible panics in BeginBock- or EndBlock-related consensus methods could cause a chain halt
if fd.IsExtension() {
panic(fmt.Errorf("proto3 declared messages do not support extensions: testpb.TestNoModuleOptionModule"))
}
panic(fmt.Errorf("message testpb.TestNoModuleOptionModule does not contain field %s", fd.FullName()))

Check warning

Code scanning / CodeQL

Panic in BeginBock or EndBlock consensus methods Warning test

Possible panics in BeginBock- or EndBlock-related consensus methods could cause a chain halt
switch fd.FullName() {
default:
if fd.IsExtension() {
panic(fmt.Errorf("proto3 declared messages do not support extensions: testpb.TestNoGoImportModule"))

Check warning

Code scanning / CodeQL

Panic in BeginBock or EndBlock consensus methods Warning test

Possible panics in BeginBock- or EndBlock-related consensus methods could cause a chain halt
if fd.IsExtension() {
panic(fmt.Errorf("proto3 declared messages do not support extensions: testpb.TestNoGoImportModule"))
}
panic(fmt.Errorf("message testpb.TestNoGoImportModule does not contain field %s", fd.FullName()))

Check warning

Code scanning / CodeQL

Panic in BeginBock or EndBlock consensus methods Warning test

Possible panics in BeginBock- or EndBlock-related consensus methods could cause a chain halt
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant