)]}'
{
  "commit": "d5158805df26f4d96501af158053d1614cc6964c",
  "tree": "fb69dfb754dc1a18ee35424d38becd4ecef97b92",
  "parents": [
    "60b2413bdfb7caa7c980422cedcb2df4e76bc244"
  ],
  "author": {
    "name": "Kazami Chan",
    "email": "ccchan@google.com",
    "time": "Tue Mar 26 17:57:34 2024 +0800"
  },
  "committer": {
    "name": "Kazami Chan",
    "email": "ccchan@google.com",
    "time": "Thu Mar 28 03:05:16 2024 +0000"
  },
  "message": "Fix FD leak for anon_inode:sync_file\n\nThe leaked file descriptor is a fence, created by dupFromAidl() within AidlCameraDeviceSession::RequestStreamBuffers(). This function duplicates the information to newly created HAL buffers.\n\nWhen it\u0027s time for the buffers to be returned (typically at the end of the capture session) within AidlCameraDeviceSession::ReturnStreamBuffers(), HAL buffers are converted back to Aidl buffers. In the original code, a new fence is directly created for the acquire_fence related to Aidl buffers, leading to the duplicated one without any calls to close it.\n\nIn this update, the behavior of acquire_fence is aligned with release_fence. We apply makeToAidlIfNotNull() to assign back to the related Aidl buffers, ensuring they are later successfully closed in the framework layer.\n\nBug: 328671068\nTest: android.hardware.cts.CameraGLTest#testCameraToSurfaceTextureMetadata\nChange-Id: I888222b1df57c4bafb65e2ddf5447c01b108f83d\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "3cf09f3142ae4d157f2a743a86127e05844ef00f",
      "old_mode": 33188,
      "old_path": "common/hal/aidl_service/aidl_utils.cc",
      "new_id": "5be472e46ec1d16755f10c5f2662b98d1d856d40",
      "new_mode": 33188,
      "new_path": "common/hal/aidl_service/aidl_utils.cc"
    }
  ]
}