commit 9c05f75eaa7d8e04c8f67c2cf6840ec9f79940d0
author Maciej Żenczykowski <maze@google.com> Thu Apr 13 09:15:01 2023 +0000
committer Maciej Żenczykowski <maze@google.com> Thu Apr 13 09:15:22 2023 +0000
tree 710ed2f70d2d8b2fec91cb1e2e4c44daf13eaf62
parent 4063f3e884f0bbdd59ac9c7d2106501d4ef5754c

clatd: only wait 15s for SIGTERM during shutdown

Test: TreeHugger
Bug: 277682237
Signed-off-by: Maciej Żenczykowski <maze@google.com>
Change-Id: If69c00f6f1c3cc1db9f8942852eeb1dbd0b4ab2e

main.c

diff --git a/main.c b/main.c
index 11da65b..f888041 100644
--- a/main.c
+++ b/main.c
@@ -194,8 +194,12 @@
 
   if (running) {
     logmsg(ANDROID_LOG_INFO, "Clatd on %s waiting for SIGTERM", uplink_interface);
-    while (running) sleep(60);
-    logmsg(ANDROID_LOG_INFO, "Clatd on %s received SIGTERM", uplink_interface);
+    // let's give higher level java code 15 seconds to kill us,
+    // but eventually terminate anyway, in case system server forgets about us...
+    // sleep() should be interrupted by SIGTERM, the handler should clear running
+    sleep(15);
+    logmsg(ANDROID_LOG_INFO, "Clatd on %s %s SIGTERM", uplink_interface,
+           running ? "timed out waiting for" : "received");
   } else {
     logmsg(ANDROID_LOG_INFO, "Clatd on %s already received SIGTERM", uplink_interface);
   }