diff --git a/test/unittest/update_client/client_unittest.cpp b/test/unittest/update_client/client_unittest.cpp
index fb8d4281ff9548089e818bb38104794709fa162a..ce3c2340df697cb724239b20f89634ec11fb11cf 100644
--- a/test/unittest/update_client/client_unittest.cpp
+++ b/test/unittest/update_client/client_unittest.cpp
@@ -73,8 +73,11 @@ public:
         size_t bytes = fread(buffer.data(), 1, JSON_MAX_SIZE, fp);
         if (bytes > 0) {
             int32_t ret = UpdateService::ParseJsonFile(buffer, info);
-            CLIENT_CHECK(ret == 0, return 0, "parse json error");
+            CLIENT_CHECK(ret == 0,
+            (void)fclose(fp);
+            return 0, "parse json error");
         }
+        (void)fclose(fp);
         return 0;
     }