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

Max evidence age not effective #3253

Closed
2 of 4 tasks
hendrikhofstadt opened this issue Jan 8, 2019 · 4 comments · Fixed by #3225
Closed
2 of 4 tasks

Max evidence age not effective #3253

hendrikhofstadt opened this issue Jan 8, 2019 · 4 comments · Fixed by #3225
Assignees

Comments

@hendrikhofstadt
Copy link
Contributor

hendrikhofstadt commented Jan 8, 2019

Currently the max evidence age in the cosmos sdk is not effective since the Time field in the ABCI evidence is propagated with the block time in which the evidence was reported.

Relevant code in Tendermint getBeginBlockValidatorInfo

	for i, ev := range block.Evidence.Evidence {
		// We need the validator set. We already did this in validateBlock.
		// TODO: Should we instead cache the valset in the evidence itself and add
		// `SetValidatorSet()` and `ToABCI` methods ?
		valset, err := LoadValidators(stateDB, ev.Height())
		if err != nil {
			panic(err) // shouldn't happen
		}
		byzVals[i] = types.TM2PB.Evidence(ev, valset, block.Time)
	}

	return commitInfo, byzVals

That allows validators to be rejailed/slashed if the evidence is older than the maxAge but within the Tendermint max evidence age.


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@hendrikhofstadt hendrikhofstadt changed the title Max evidence ago not effective Max evidence age not effective Jan 8, 2019
@nodebreaker0-0
Copy link

With this, I am in the fourth prison.

@alexanderbez alexanderbez self-assigned this Jan 8, 2019
@jackzampolin
Copy link
Member

This will be closed by #3225

@sunnya97
Copy link
Member

@nodebreaker-carl Don't worry, with the tombstone fix (#3103 / #3225), you'll just never get to get out of jail in the first place! 😅

@nodebreaker0-0
Copy link

@sunnya97 Oh, yes, there's a period. Captain

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants