run: Don't kill children twice.
authorAndre Noll <maan@tuebingen.mpg.de>
Tue, 14 Nov 2017 03:12:02 +0000 (04:12 +0100)
committerAndre Noll <maan@tuebingen.mpg.de>
Sun, 19 Nov 2017 01:53:33 +0000 (02:53 +0100)
commit45ae697d187c12a7ed40ae71b6a8adeda4b32b6c
tree476417457bbfc94361ba1cb587daf4c239d77949
parent7a6ff706b99f35a3f4f1ee116f49827dbdf15c64
run: Don't kill children twice.

When handle_signal(), the signal dispatcher of the run subcommand,
detects that SIGINT or SIGTERM was received, it calls kill_children()
to terminate any running rsync or rm processes. It then returns
negative which terminates the select loop. However, after select_loop()
returns, kill_children() is called again. Also the error message is
logged twice.

Not a biggie, but let's get rid of this redundancy by removing the
first call to kill_children().

Since handle_signal() is only called from com_run(), this patch
affects only the run subcommand.
dss.c