diff --git a/weed/filesys/meta_cache/meta_cache_subscribe.go b/weed/filesys/meta_cache/meta_cache_subscribe.go
index 409120060..19f5e398a 100644
--- a/weed/filesys/meta_cache/meta_cache_subscribe.go
+++ b/weed/filesys/meta_cache/meta_cache_subscribe.go
@@ -66,4 +66,5 @@ func SubscribeMetaEvents(mc *MetaCache, selfSignature int32, client filer_pb.Fil
 		return true
 	})
 
+	return nil
 }
diff --git a/weed/util/retry.go b/weed/util/retry.go
index 4b34d5129..569f8fda1 100644
--- a/weed/util/retry.go
+++ b/weed/util/retry.go
@@ -35,7 +35,7 @@ func Retry(name string, job func() error) (err error) {
 func RetryForever(name string, job func() error, onErrFn func(err error) bool) {
 	waitTime := time.Second
 	for {
-		err = job()
+		err := job()
 		if err == nil {
 			break
 		}