X-Git-Url: https://git.sesse.net/?p=cubemap;a=blobdiff_plain;f=main.cpp;h=50fd48e729f1df4c47b30755e017b9f2d2c1a0d8;hp=fc36861f64ab827d22b988f387e1a2b2e9cba75b;hb=HEAD;hpb=20e85bd6901355cc40a6cfb4c0deb7232d9aa63f

diff --git a/main.cpp b/main.cpp
index fc36861..50fd48e 100644
--- a/main.cpp
+++ b/main.cpp
@@ -1,5 +1,6 @@
 #include <assert.h>
 #include <errno.h>
+#include <fcntl.h>
 #include <getopt.h>
 #include <limits.h>
 #include <signal.h>
@@ -15,6 +16,7 @@
 #include <map>
 #include <set>
 #include <string>
+#include <unordered_map>
 #include <utility>
 #include <vector>
 
@@ -445,7 +447,7 @@ start:
 		serialize_start.tv_nsec = loaded_state.serialize_start_usec() * 1000ull;
 
 		// Deserialize the streams.
-		map<string, string> stream_headers_for_url;  // See below.
+		unordered_map<string, string> stream_headers_for_url;  // See below.
 		for (const StreamProto &stream : loaded_state.streams()) {
 			if (all_urls.count(stream.url()) == 0) {
 				// Delete the stream backlogs.
@@ -520,7 +522,9 @@ start:
 	     loaded_state.mutable_clients()->end(),
 	     OrderByConnectionTime());
 	for (int i = 0; i < loaded_state.clients_size(); ++i) {
-		if (all_urls.count(loaded_state.clients(i).url()) == 0) {
+		if (!loaded_state.clients(i).url().empty() &&
+		    all_urls.count(loaded_state.clients(i).url()) == 0) {
+			// Belongs to a dead stream (not keepalive), so we just have to close.
 			safe_close(loaded_state.clients(i).sock());
 		} else {
 			servers->add_client_from_serialized(loaded_state.clients(i), short_response_pool);
@@ -529,6 +533,15 @@ start:
 	
 	short_response_pool.clear();  // No longer needed; the clients have their own refcounts now.
 
+	// Put back the HLS zombies. There's no really good allocation here
+	// except round-robin; it would be marginally more efficient to match it
+	// to the client (since that would have them deleted immediately when
+	// the client requests the next fragment, instead of being later weeded
+	// out during statistics collection), but it's not a big deal.
+	for (const HLSZombieProto &zombie_proto : loaded_state.hls_zombies()) {
+		servers->add_hls_zombie_from_serialized(zombie_proto);
+	}
+
 	servers->run();
 
 	// Now delete all inputs that are longer in use, and start the others.
@@ -610,6 +623,9 @@ start:
 	}
 	for (const auto &key_and_input_with_refcount : inputs) {
 		key_and_input_with_refcount.second.input->stop();
+		if (stopped) {
+			key_and_input_with_refcount.second.input->close_socket();
+		}
 	}
 	servers->stop();
 
@@ -649,6 +665,10 @@ start:
 	char buf[16];
 	sprintf(buf, "%d", state_fd);
 
+	// Unset the close-on-exec flag for the state fd.
+	// (This can't leak into a child, since there's only one thread left.)
+	fcntl(state_fd, F_SETFD, 0);
+
 	for ( ;; ) {
 		execlp(argv0_canon, argv0_canon, config_filename_canon, "--state", buf, nullptr);
 		open_logs(config.log_destinations);