Merge pull request #10462 from zfox23/fixIdentityUpdatedAt

Force _identityUpdatedAt to stay above 0, fixing more identity packet bugs
This commit is contained in:
Zach Fox 2017-05-15 15:44:26 -07:00 committed by GitHub
commit 64ae5119ae

View file

@ -1538,7 +1538,14 @@ void AvatarData::processAvatarIdentity(const Identity& identity, bool& identityC
// use the timestamp from this identity, since we want to honor the updated times in "server clock"
// this will overwrite any changes we made locally to this AvatarData's _identityUpdatedAt
_identityUpdatedAt = identity.updatedAt - clockSkew;
// Additionally, ensure that the timestamp that we try to record isn't negative, as
// "_identityUpdatedAt" is an *unsigned* 64-bit integer. Furthermore, negative timestamps
// wouldn't make sense.
if (identity.updatedAt > clockSkew) {
_identityUpdatedAt = identity.updatedAt - clockSkew;
} else {
_identityUpdatedAt = 0;
}
}
QByteArray AvatarData::identityByteArray() const {