Wrong if check condition, error handling logic never trigger forever

[zyfrank]

union/galoisd/grpc/server.go

Lines 308 to 313 in f15673c

	if len(req.TrustedCommit.Signatures) > len(req.TrustedCommit.Signatures) {
	return nil, fmt.Errorf("More signatures than validators")
	}
	if len(req.UntrustedCommit.Signatures) > len(req.UntrustedCommit.Signatures) {
	return nil, fmt.Errorf("More signatures than validators")
	}

should change to
if len(req.TrustedCommit.Signatures) > len(req.TrustedCommit.Validators)

and

if len(req.UntrustedCommit.Signatures) > len(req.UntrustedCommit.Validators)

[heisdml]

you right

[bit2swaz]

im picking this up. confirmed both checks at server.go:308/:311 compare Signatures length against itself, so the guard is dead. and the same copy paste bug is also in bls12381_server.go:108/:111. im fixing both to compare against Validators and adding a regression test for Poll