Properly invalidate create_pid also for the post-create hook.

If the process associated with the create_pid dies, handle_sigchld()
investigates snapshot_creation_status to tell whether the pre-create
hook, the rsync process or the post-create hook has died.

In the first two cases, handle_pre_create_hook_exit() and
handle_rsync_exit() are called, respectively. Both functions correctly
invalidate create_pid (by resetting it to zero).

However, the post-create hook handling code misses to reset
create_pid. This causes dss to send SIGTERM to this pid on exit,
which might be fatal as the pid might have been reassigned to some
unrelated process in the meanwhile.

Fix this bug by moving the invalidation of create_pid to the end of
the "if (pid == create_pid)" clause, which even saves a line of code.

Many thanks to Sebastian Stark who pointed out that bug.

diff --git a/dss.c b/dss.c
index c5f636c2ed8b86ad5170c164d3f6e2d4d08d862c..93ab802cb123bddde776042a9a9cff266102091c 100644 (file)
--- a/dss.c
+++ b/dss.c
@@ -734,7 +734,6 @@ static int handle_rsync_exit(int status)
        free(name_of_reference_snapshot);
        name_of_reference_snapshot = NULL;
 out:
        free(name_of_reference_snapshot);
        name_of_reference_snapshot = NULL;
 out:

-       create_pid = 0;

        create_process_stopped = 0;
        return ret;
 }
        create_process_stopped = 0;
        return ret;
 }


@@ -766,7 +765,6 @@ static int handle_pre_create_hook_exit(int status)
        snapshot_creation_status = HS_PRE_SUCCESS;
        ret = 1;
 out:
        snapshot_creation_status = HS_PRE_SUCCESS;
        ret = 1;
 out:

-       create_pid = 0;

        return ret;
 }
 
        return ret;
 }
 


@@ -792,6 +790,7 @@ static int handle_sigchld(void)
                                snapshot_creation_status);
                        return -E_BUG;
                }
                                snapshot_creation_status);
                        return -E_BUG;
                }

+               create_pid = 0;

        }
        if (pid == remove_pid) {
                ret = handle_remove_exit(status);
        }
        if (pid == remove_pid) {
                ret = handle_remove_exit(status);