Disallow negative values for Tendermint client state duration parameters

[hu55a1n1]

Summary of Bug

I think we're currently allowing negative values for TrustingPeriod, UnbondingPeriod and MaxClockDrift while the spec forbids negative values and uses uint64 to represent them ->

ibc-go/modules/light-clients/07-tendermint/client_state.go

Lines 128 to 136 in f693f6c

	if cs.TrustingPeriod == 0 {
	return sdkerrors.Wrap(ErrInvalidTrustingPeriod, "trusting period cannot be zero")
	}
	if cs.UnbondingPeriod == 0 {
	return sdkerrors.Wrap(ErrInvalidUnbondingPeriod, "unbonding period cannot be zero")
	}
	if cs.MaxClockDrift == 0 {
	return sdkerrors.Wrap(ErrInvalidMaxClockDrift, "max clock drift cannot be zero")
	}

Expected Behaviour

Disallow negative values for these parameters.

Version

v5.0.0

---

For Admin Use

• Not duplicate issue
• Appropriate labels applied
• Appropriate contributors tagged/assigned