move the general rules cleanup to pod level

Signed-off-by: Xu Wang <[email protected]>

Author: gnawux

src/init.c

@@ -553,6 +553,7 @@ static void hyper_flush_channel()
 
 void hyper_pod_destroyed(struct hyper_pod *pod, int failed)
 {
+	hyper_cleanup_portmapping(pod);
 	hyper_cleanup_mounts(pod);
 	hyper_ctl_append_msg(&hyper_epoll.ctl, failed?ERROR:ACK, NULL, 0);
 	// Todo: this doesn't make sure peer receives the data

src/portmapping.c

@@ -258,14 +258,15 @@ int hyper_setup_container_portmapping(struct hyper_container *c, struct hyper_po
 	return 0;
 }
 
-void hyper_cleanup_container_portmapping(struct hyper_container *c, struct hyper_pod *pod)
+// cleanup pod level portmapping configurations
+void hyper_cleanup_portmapping(struct hyper_pod *pod)
 {
 	if (pod->portmap_white_lists == NULL || (pod->portmap_white_lists->i_num == 0 &&
 			pod->portmap_white_lists->e_num == 0)) {
 		return;
 	}
 
-	int i = 0, j = 0;
+	int j = 0;
 	char rule[128] = {0};
 	for (j=0; j<pod->portmap_white_lists->i_num; j++) {
 		sprintf(rule, "-s %s -j ACCEPT",
@@ -280,7 +281,17 @@ void hyper_cleanup_container_portmapping(struct hyper_container *c, struct hyper
 			fprintf(stderr, "cleanup accept_rule '%s' failed\n", rule);
 		}
 	}
+}
 
+void hyper_cleanup_container_portmapping(struct hyper_container *c, struct hyper_pod *pod)
+{
+	if (pod->portmap_white_lists == NULL || (pod->portmap_white_lists->i_num == 0 &&
+			pod->portmap_white_lists->e_num == 0)) {
+		return;
+	}
+
+	int i = 0, j = 0;
+	char rule[128] = {0};
 	if (c->ports_num == 0) {
 		return;
 	}

src/portmapping.h

@@ -17,6 +17,7 @@ struct hyper_pod;
 struct hyper_container;
 int hyper_setup_portmapping(struct hyper_pod *pod);
 int hyper_setup_container_portmapping(struct hyper_container *c, struct hyper_pod *pod);
+void hyper_cleanup_portmapping(struct hyper_pod *pod);
 void hyper_cleanup_container_portmapping(struct hyper_container *c, struct hyper_pod *pod);
 
 #endif