From 16e95f33b73b21dc77c7aab40b9764ecc38382a5 Mon Sep 17 00:00:00 2001
From: Pedro Pombeiro <PombeirP@users.noreply.github.com>
Date: Wed, 19 Sep 2018 17:44:30 +0200
Subject: [PATCH] whisper: Fix interpretation of `to` parameter in
`shh_requestMessages` (#16996)
The argument is inclusive rather than exclusive, according to docs.
---
whisper/mailserver/mailserver.go | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/whisper/mailserver/mailserver.go b/whisper/mailserver/mailserver.go
index d32eaddec..af9418d9f 100644
--- a/whisper/mailserver/mailserver.go
+++ b/whisper/mailserver/mailserver.go
@@ -118,7 +118,7 @@ func (s *WMailServer) processRequest(peer *whisper.Peer, lower, upper uint32, bl
var err error
var zero common.Hash
kl := NewDbKey(lower, zero)
- ku := NewDbKey(upper, zero)
+ ku := NewDbKey(upper+1, zero) // LevelDB is exclusive, while the Whisper API is inclusive
i := s.db.NewIterator(&util.Range{Start: kl.raw, Limit: ku.raw}, nil)
defer i.Release()
--
GitLab