commit 0372f296b515f90dde1120bebd24e405a7a75525
author Maciej Żenczykowski <maze@google.com> Fri Mar 14 19:21:00 2025 -0700
committer Maciej Żenczykowski <maze@google.com> Mon Mar 17 22:53:38 2025 -0700
tree 9853c672e695a597e8c3fd46366e50ebd802f7cb
parent b984f31e55cf3fdcee6621c7a25a3606e6aee923

clatd: one less shutdown log line

less spam

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

clatd/main.c

diff --git a/clatd/main.c b/clatd/main.c
index 1d2b227..fca996b 100644
--- a/clatd/main.c
+++ b/clatd/main.c
@@ -201,13 +201,11 @@
 
   event_loop(&tunnel);
 
-  logmsg(ANDROID_LOG_INFO, "Shutting down clat on %s", uplink_interface);
-
   if (sigterm) {
-    logmsg(ANDROID_LOG_INFO, "Clatd on %s already received SIGTERM", uplink_interface);
+    logmsg(ANDROID_LOG_INFO, "Shutting down clatd on %s, already received SIGTERM", uplink_interface);
   } else {
     // this implies running == false, ie. we received EOF or ENETDOWN error.
-    logmsg(ANDROID_LOG_INFO, "Clatd on %s waiting for SIGTERM", uplink_interface);
+    logmsg(ANDROID_LOG_INFO, "Shutting down clatd on %s, waiting for 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 set 'sigterm'