diff --git a/pkg/common/perf/perf.go b/pkg/common/perf/perf.go
index 68fe6e3fe150a1817e5397938290a2673d22e210..691abc775036719bd8064cdb111cc3fa67a1dc3d 100644
--- a/pkg/common/perf/perf.go
+++ b/pkg/common/perf/perf.go
@@ -256,15 +256,12 @@ func (p *perf) Destroy() {
 // CgroupStat report perf stat for cgroup
 func CgroupStat(cgpath string, dur time.Duration) (*Stat, error) {
 	p, err := newPerf(cgpath)
-	defer func() {
-		if p != nil {
-			p.Destroy()
-		}
-	}()
 	if err != nil {
 		return nil, errors.Errorf("perf init failed: %v", err)
 	}
 
+	defer p.Destroy()
+
 	if err := p.Start(); err != nil {
 		return nil, errors.Errorf("perf start failed: %v", err)
 	}
diff --git a/pkg/common/util/file.go b/pkg/common/util/file.go
index 5a67b72ed8ecca264915d7560a5a394d87ddae29..08e9acda5baa41844e167aba1c26d2463ca28683 100644
--- a/pkg/common/util/file.go
+++ b/pkg/common/util/file.go
@@ -140,14 +140,12 @@ func AppendFile(path, content string) error {
 		return fmt.Errorf("%v is not a file", path)
 	}
 	f, err := os.OpenFile(path, os.O_APPEND|os.O_CREATE|os.O_WRONLY, constant.DefaultFileMode)
-	defer func() {
-		if err != f.Close() {
-			return
-		}
-	}()
 	if err != nil {
 		return fmt.Errorf("error opening file: %v", err)
 	}
+
+	defer f.Close()
+
 	if _, err := f.WriteString(content); err != nil {
 		return fmt.Errorf("error writing file: %v", err)
 	}