Blame SOURCES/rhnsd-5.0.13-5-el7-to-rhnsd-5.0.13-6-el7.patch

baeaa7
diff --git a/rhnsd.c b/rhnsd.c
baeaa7
index 5f0711d..ad2100d 100644
baeaa7
--- a/rhnsd.c
baeaa7
+++ b/rhnsd.c
baeaa7
@@ -49,6 +49,9 @@ static int write_pid (const char *file);
baeaa7
 static void print_version (FILE *stream, struct argp_state *state);
baeaa7
 void (*argp_program_version_hook) (FILE *, struct argp_state *) = print_version;
baeaa7
 
baeaa7
+/* Current rhn_check pid */
baeaa7
+pid_t rhn_check_pid;
baeaa7
+
baeaa7
 /* Definitions of arguments for argp functions.  */
baeaa7
 static const struct argp_option options[] =
baeaa7
 {
baeaa7
@@ -145,16 +148,35 @@ int main (int argc, char **argv)
baeaa7
     if (!foreground) {
baeaa7
 	int i;
baeaa7
 
baeaa7
-	if (fork ())
baeaa7
-	    exit (0);
baeaa7
+	    pid_t pid;
baeaa7
+        pid = fork();
baeaa7
+        if (pid < 0)
baeaa7
+	        exit (EXIT_FAILURE);
baeaa7
+
baeaa7
+        if (pid > 0)
baeaa7
+            exit(EXIT_SUCCESS);
baeaa7
+
baeaa7
 
baeaa7
 	for (i = 0; i < getdtablesize(); i++)
baeaa7
 	    close (i);
baeaa7
 
baeaa7
-	if (fork ())
baeaa7
-	    exit (0);
baeaa7
+	/* read from /dev/null */
baeaa7
+	int devnull = open("/dev/null", O_RDONLY);
baeaa7
+	if ((devnull >= 0) && (devnull != STDIN_FILENO)) {
baeaa7
+	    dup2(devnull, STDIN_FILENO);
baeaa7
+	    close(devnull);
baeaa7
+	}
baeaa7
+
baeaa7
+        pid = fork();
baeaa7
+        if (pid < 0)
baeaa7
+            exit (EXIT_FAILURE);
baeaa7
+
baeaa7
+        if (pid > 0)
baeaa7
+            exit(EXIT_SUCCESS);
baeaa7
 
baeaa7
-	setsid();
baeaa7
+        if (setsid() < 0)
baeaa7
+            exit(EXIT_FAILURE);
baeaa7
+	umask(0);
baeaa7
 
baeaa7
 	chdir ("/");
baeaa7
 
baeaa7
@@ -252,11 +274,13 @@ Written by %s.\n\
baeaa7
 static void termination_handler (int signum)
baeaa7
 {
baeaa7
     syslog(LOG_NOTICE, "Exiting");
baeaa7
-
baeaa7
+    /* Send SIGTERM to current running rhn_check
baeaa7
+     * and wait for it to exit. */
baeaa7
+    kill(rhn_check_pid, SIGTERM);
baeaa7
+    wait(NULL);
baeaa7
     /* Clean up pid file.  */
baeaa7
     unlink (_PATH_RHNDPID);
baeaa7
-
baeaa7
-    exit (EXIT_SUCCESS);
baeaa7
+    exit(EXIT_SUCCESS);
baeaa7
 }
baeaa7
 
baeaa7
 /* Returns 1 if the process in pid file FILE is running, 0 if not.  */
baeaa7
@@ -383,10 +407,11 @@ static int rhn_do_action(void)
baeaa7
 	dup2(STDOUT_FILENO, STDERR_FILENO);
baeaa7
 
baeaa7
 	/* syslog for safekeeping */
baeaa7
-	syslog(LOG_DEBUG, "running program %s", RHN_CHECK);
baeaa7
+	syslog(LOG_NOTICE, "running program %s with PID %d", RHN_CHECK, getpid());
baeaa7
 
baeaa7
         unset_signal_handlers ();
baeaa7
         /* exec binary helper */
baeaa7
+        rhn_check_pid = getpid();
baeaa7
         args[0] = RHN_CHECK;
baeaa7
         execv(RHN_CHECK, args);
baeaa7
 
baeaa7
@@ -396,6 +421,7 @@ static int rhn_do_action(void)
baeaa7
                strerror(errno));
baeaa7
         exit(errno);
baeaa7
     } else if (child > 0) {
baeaa7
+        set_signal_handlers();
baeaa7
 	int ret = 1;
baeaa7
 	char *buf, buffer[10];
baeaa7
 	int bufsize = 0;
baeaa7
diff --git a/rhnsd.init b/rhnsd.init
baeaa7
index ee3e811..dead8bf 100644
baeaa7
--- a/rhnsd.init
baeaa7
+++ b/rhnsd.init
baeaa7
@@ -53,7 +53,7 @@ start() {
baeaa7
 
baeaa7
 stop() {
baeaa7
     echo -n $"Stopping Spacewalk Daemon: "
baeaa7
-    killproc rhnsd
baeaa7
+    killproc rhnsd -TERM
baeaa7
     RETVAL=$?
baeaa7
     echo
baeaa7
     if [ $RETVAL -eq 0 ]; then
baeaa7
diff --git a/rhnsd.spec b/rhnsd.spec
baeaa7
index ee6085f..f893a8d 100644
baeaa7
--- a/rhnsd.spec
baeaa7
+++ b/rhnsd.spec
baeaa7
@@ -5,7 +5,7 @@ Source0: https://fedorahosted.org/releases/s/p/spacewalk/%{name}-%{version}.tar.
baeaa7
 URL:     https://fedorahosted.org/spacewalk
baeaa7
 Name: rhnsd
baeaa7
 Version: 5.0.13
baeaa7
-Release: 5%{?dist}
baeaa7
+Release: 6%{?dist}
baeaa7
 BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
baeaa7
 
baeaa7
 BuildRequires: gettext
baeaa7
@@ -138,6 +138,10 @@ rm -fr $RPM_BUILD_ROOT
baeaa7
 %doc LICENSE
baeaa7
 
baeaa7
 %changelog
baeaa7
+* Mon Mar 06 2017 Gennadii Altukhov <galt@redhat.com> 5.0.13-6
baeaa7
+- 1260527 RHEL7 rhnsd reload doesn't work (jdostal@redhat.com)
baeaa7
+- 1260527,1409570 - Reboot loop - rhn_check, rhnsd (jdostal@redhat.com)
baeaa7
+
baeaa7
 * Thu Jun 25 2015 Grant Gainey 5.0.13-5
baeaa7
 - Bump release to get past weirdness in brew-tags (ggainey@redhat.com)
baeaa7
 - Related: #1138939 - up2date and systemid files are managed by rhnsd itself,