Mercurial > hg
diff rust/hg-core/src/revlog/changelog.rs @ 50746:124c44b5cfad stable
rust-revlog: fix RevlogEntry.data() for NULL_REVISION
Before this change, the pseudo-entry returned by `Revlog.get_entry` for
`NULL_REVISION` would trigger errors in application code using it.
For example, this fixes a crash spotted with changelog data
while implementing RHGitaly: `Changelog.data_for_rev(-1)` was already
returning the pseudo content as expected, e.g., for `hg log`, but
`Changelog.entry_for_rev(-1).data()` would still crash with
"corrupted revlog, hash check failed for revision -1". There is
an added test for this scenario.
author | Georges Racinet <georges.racinet@octobus.net> |
---|---|
date | Thu, 06 Jul 2023 11:53:40 +0200 |
parents | 071a6c1d291e |
children | 9865af7191d2 |
line wrap: on
line diff
--- a/rust/hg-core/src/revlog/changelog.rs Thu Jul 06 11:43:26 2023 +0200 +++ b/rust/hg-core/src/revlog/changelog.rs Thu Jul 06 11:53:40 2023 +0200 @@ -336,6 +336,11 @@ changelog.data_for_rev(NULL_REVISION)?, ChangelogRevisionData::null() ); + // same with the intermediate entry object + assert_eq!( + changelog.entry_for_rev(NULL_REVISION)?.data()?, + ChangelogRevisionData::null() + ); Ok(()) } }