From 6999bbb47bc6173a6e0d04cc62efaff4633d03e6 Mon Sep 17 00:00:00 2001 From: Iwasaki Yudai Date: Mon, 8 Jan 2018 11:21:53 -0800 Subject: [PATCH] mvcc: check null before set FillPercent not to panic Since CreateBucketIfNotExists() can return nil when it gets an error, accessing FillPercent must be done after a nil check, not to cause a panic. --- mvcc/backend/backend.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mvcc/backend/backend.go b/mvcc/backend/backend.go index 87edd25f427..9d053337f78 100644 --- a/mvcc/backend/backend.go +++ b/mvcc/backend/backend.go @@ -370,10 +370,10 @@ func defragdb(odb, tmpdb *bolt.DB, limit int) error { } tmpb, berr := tmptx.CreateBucketIfNotExists(next) - tmpb.FillPercent = 0.9 // for seq write in for each if berr != nil { return berr } + tmpb.FillPercent = 0.9 // for seq write in for each b.ForEach(func(k, v []byte) error { count++