hak5
/
overseer
mirror of https://github.com/hak5/overseer.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

If fetcher is disabled, don't test to move files 

If the user only wants graceful restarts from this library, and the user running the go program doesn't have write access to the path to the binary, overseer fails to start with an error 

`[overseer] disabled. run failed: cannot move binary (exit status 1)`

This commit checks if fetching is enabled, if not, it avoids the move check
master
Diego Medina 2017-07-26 07:48:54 -04:00 committed by GitHub
parent 8dea607db4
commit 75c69b4823
1 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
proc_master.go
View File

@ -90,11 +90,13 @@ func (mp *master) checkBinary() error {
mp.binHash = hash.Sum(nil)
f.Close()
//test bin<->tmpbin moves
if err := move(tmpBinPath, mp.binPath); err != nil {
return fmt.Errorf("cannot move binary (%s)", err)
}
if err := move(mp.binPath, tmpBinPath); err != nil {
return fmt.Errorf("cannot move binary back (%s)", err)
if mp.Config.Fetcher != nil {
if err := move(tmpBinPath, mp.binPath); err != nil {
return fmt.Errorf("cannot move binary (%s)", err)
}
if err := move(mp.binPath, tmpBinPath); err != nil {
return fmt.Errorf("cannot move binary back (%s)", err)
}
}
return nil
}