diamondburned/arikawa
1
0
Fork 0
mirror of https://github.com/diamondburned/arikawa.git synced 2025-03-22 09:59:37 +00:00
Code Issues Releases Activity

state: Fix out of bound panic in Messages (#235)

Browse source
This commit is contained in:
Maximilian von Lindern 2021-06-25 07:46:27 +02:00 committed by GitHub
parent 6c349d9750
commit 8e4e48e25e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
state/state.go
View file

@ -700,7 +700,12 @@ func (s *State) Messages(channelID discord.ChannelID, limit uint) ([]discord.Mes
if s.tracksMessage(&apiMessages[0]) && len(storeMessages) < s.MaxMessages() { if s.tracksMessage(&apiMessages[0]) && len(storeMessages) < s.MaxMessages() {
// Only add as many messages as the store can hold. // Only add as many messages as the store can hold.
for _, m := range apiMessages[:s.MaxMessages()-len(storeMessages)] { i := s.MaxMessages() - len(storeMessages)
if i > len(apiMessages) {
i = len(apiMessages)
}
for _, m := range apiMessages[:i] {
if err := s.Cabinet.MessageSet(m, false); err != nil { if err := s.Cabinet.MessageSet(m, false); err != nil {
return nil, err return nil, err
} }