commit b1f89da5c3cd672848c8ef5c427ff813e0b5eaf8
author Treehugger Robot <android-test-infra-autosubmit@system.gserviceaccount.com> Thu Jun 13 01:58:37 2024 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Thu Jun 13 01:58:37 2024 +0000
tree d7e9115dd717f9d5768a0743312167c7531ca9dd
parent 2ba21d58c7f4fa0974187524fca645186f968bbe
parent b62d79c423570421e9813bf0db2dbd7c981a3c71

Merge "Add individual try/catch blocks for ParcelFileDescriptor closure" into main am: b62d79c423

Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/3124594

Change-Id: I5dec3a2aa6eddad15f2e7f2a6d195419388a60dc
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

service/src/com/android/server/connectivity/ClatCoordinator.java

diff --git a/service/src/com/android/server/connectivity/ClatCoordinator.java b/service/src/com/android/server/connectivity/ClatCoordinator.java
index d886182..f333dae 100644
--- a/service/src/com/android/server/connectivity/ClatCoordinator.java
+++ b/service/src/com/android/server/connectivity/ClatCoordinator.java
@@ -683,13 +683,25 @@
             throw new IOException("Failed to start clat ", e);
         } finally {
             if (tunFd != null) {
-                tunFd.close();
+                try {
+                    tunFd.close();
+                } catch (IOException e) {
+                    Log.e(TAG, "Fail to close tun file descriptor " + e);
+                }
             }
             if (readSock6 != null) {
-                readSock6.close();
+                try {
+                    readSock6.close();
+                } catch (IOException e) {
+                    Log.e(TAG, "Fail to close read socket " + e);
+                }
             }
             if (writeSock6 != null) {
-                writeSock6.close();
+                try {
+                    writeSock6.close();
+                } catch (IOException e) {
+                    Log.e(TAG, "Fail to close write socket " + e);
+                }
             }
         }
     }