[maemo-commits] [maemo-commits] r18623 - in projects/haf/trunk/ke-recv: debian src

[From: subversion at stage.maemo.org]

Author: kihamala
Date: 2009-06-05 13:45:17 +0300 (Fri, 05 Jun 2009)
New Revision: 18623

Modified:
   projects/haf/trunk/ke-recv/debian/changelog
   projects/haf/trunk/ke-recv/src/ke-recv.c
   projects/haf/trunk/ke-recv/src/mmc-check.c
Log:
Fixes: NB#119586 - Unsafe signal handler in ke-recv


Modified: projects/haf/trunk/ke-recv/debian/changelog
===================================================================
--- projects/haf/trunk/ke-recv/debian/changelog	2009-06-05 10:35:18 UTC (rev 18622)
+++ projects/haf/trunk/ke-recv/debian/changelog	2009-06-05 10:45:17 UTC (rev 18623)
@@ -1,6 +1,6 @@
 ke-recv (3.19-1~unreleased) unstable; urgency=low
 
-  * foo
+  * Fixes: NB#119586 - Unsafe signal handler in ke-recv
 
  -- Kimmo Hämäläinen <kimmo.hamalainen at nokia.com>  Fri,  5 Jun 2009 13:33:23 +0300
 

Modified: projects/haf/trunk/ke-recv/src/ke-recv.c
===================================================================
--- projects/haf/trunk/ke-recv/src/ke-recv.c	2009-06-05 10:35:18 UTC (rev 18622)
+++ projects/haf/trunk/ke-recv/src/ke-recv.c	2009-06-05 10:45:17 UTC (rev 18623)
@@ -3031,7 +3031,6 @@
 
 static void sigterm(int signo)
 {
-        ULOG_INFO_L("got SIGTERM");
         g_main_loop_quit(mainloop);
 }
 

Modified: projects/haf/trunk/ke-recv/src/mmc-check.c
===================================================================
--- projects/haf/trunk/ke-recv/src/mmc-check.c	2009-06-05 10:35:18 UTC (rev 18622)
+++ projects/haf/trunk/ke-recv/src/mmc-check.c	2009-06-05 10:45:17 UTC (rev 18623)
@@ -49,15 +49,11 @@
         int child_status = -1;
         assert(signo == SIGCHLD);
 
-        ULOG_DEBUG_F("child pid=%d", child_pid);
         ret = waitpid(child_pid, &child_status, 0);
         if (ret == -1) {
-                ULOG_ERR_F("waitpid() failed");
                 exit(1);
         }
         if (WIFEXITED(child_status)) {
-                ULOG_DEBUG_F("child returned: %d",
-                             WEXITSTATUS(child_status));
                 if (WEXITSTATUS(child_status) == 1) {
                         exit(3);
                 } else if (WEXITSTATUS(child_status) == 2) {
@@ -65,7 +61,6 @@
                 }
                 exit(0);
         } else {
-                ULOG_WARN_F("child terminated abnormally");
                 exit(2);
         }
 }