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.
etcd-io · Jan 8, 2018 · 1ae0c0b · 1ae0c0b
1 parent ec43197
commit 1ae0c0b
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/mvcc/backend/backend.go b/mvcc/backend/backend.go
@@ -373,10 +373,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++