yugabyte/yugabyte-db

[Platform] Leaks of resources

Open

#6,586 opened on 2020年12月8日

GitHub で見る
 (7 comments) (0 reactions) (0 assignees)C (1,003 forks)batch import
area/platformgood first issue

Repository metrics

Stars
 (8,229 stars)
PR merge metrics
 (平均マージ 17d 21h) (30d で 92 merged PRs)

説明

  1. UniverseTaskBase.java, isServerAlive(), absent ybService.closeClient();
  2. Universe.java, getMasterLeader(), it is better to put closeClient() into the finally block;
  3. SetFlagInMemory.java, run(), closeClient() could be missed in case of any exceptions raised after getClient() and before closeClient();
  4. WaitForServerReady.java, run(), it is better to put closeClient() into the finally block. Yes, it is almost impossible that ServerSubTaskBase.getHostPort() will throw an exception but actually this probability is not 0;
  5. SessionController.java, getLogs(), Resource leak: 'reader' is never closed;
  6. SwamperHelper.java, writeTargetJsonFile(), the file may stay opened (in case of file.write of file.flush throws an exception);
  7. CertificateHelper.java, createRootCA(), Resource leak: 'certWriter' is never closed, Resource leak: 'keyWriter' is never closed; createClientCertificate(), 'clientCertWriter' and 'clientKeyWriter' are never closed (in case when storagePath != null).

コントリビューターガイド