Move "shutdown protocol" after other uses of 'defer'

Currently, `memcachedClient.Stop()` is getting called on return from
`main()`, which breaks things if the cache moves address.

cmd/fluxd/main.go

@@ -222,15 +222,6 @@ func main() {
 		errc <- fmt.Errorf("%s", <-c)
 	}()
 
-	// This means we can return, and it will use the shutdown
-	// protocol.
-	defer func() {
-		// wait here until stopping.
-		logger.Log("exiting", <-errc)
-		close(shutdown)
-		shutdownWg.Wait()
-	}()
-
 	// Cluster component.
 	var clusterVersion string
 	var sshKeyRing ssh.KeyRing
@@ -546,5 +537,15 @@ func main() {
 		}()
 	}
 
+	// This means we can return, and it will use the shutdown
+	// protocol.
+	// THIS MUST COME AFTER ALL OTHER CALLS TO defer
+	defer func() {
+		// wait here until stopping.
+		logger.Log("exiting", <-errc)
+		close(shutdown)
+		shutdownWg.Wait()
+	}()
+
 	// Fall off the end, into the waiting procedure.
 }