Skip to content

Fix rank overflow in zslInsert with more than 2B entries.#9249

Merged
oranagra merged 1 commit intoredis:unstablefrom
enjoy-binbin:unsigned_long_rank
Jul 18, 2021
Merged

Fix rank overflow in zslInsert with more than 2B entries.#9249
oranagra merged 1 commit intoredis:unstablefrom
enjoy-binbin:unsigned_long_rank

Conversation

@enjoy-binbin
Copy link
Contributor

If there are more than 2B entries in a zset.
The calculated span will overflow.

If there are more than 2B entries in a zset.
The calculated span will overflow.
@oranagra oranagra changed the title Fix rank overflow in zslInsert. Fix rank overflow in zslInsert with more than 2B entries. Jul 18, 2021
@oranagra oranagra added the release-notes indication that this issue needs to be mentioned in the release notes label Jul 18, 2021
@oranagra
Copy link
Member

should this be mention in the release notes, i'm not sure realistically poses an issue for anyone..

@oranagra oranagra merged commit e7ccdee into redis:unstable Jul 18, 2021
@enjoy-binbin enjoy-binbin deleted the unsigned_long_rank branch July 18, 2021 13:49
@yangbodong22011
Copy link
Contributor

Resolves #8043

@oranagra
Copy link
Member

thanks @yangbodong22011 i remembered i saw that mentioned somewhere, but couldn't find it..

JackieXie168 pushed a commit to JackieXie168/redis that referenced this pull request Sep 8, 2021
If there are more than 2B entries in a zset.
The calculated span will overflow.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-notes indication that this issue needs to be mentioned in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants