--- mqtt/src/daemon.c	2012/04/25 13:33:53	1.2.2.13
+++ mqtt/src/daemon.c	2012/05/05 13:39:27	1.2.2.24
@@ -5,10 +5,6 @@
 #include "mqttd_calls.h"
 
 
-static void *startSession(sched_task_t *task);
-static pthread_attr_t attr;
-
-
 static inline struct tagSession *
 initSession(int sock, ait_val_t * __restrict v)
 {
@@ -23,7 +19,6 @@ initSession(int sock, ait_val_t * __restrict v)
 	sess = malloc(sizeof(struct tagSession));
 	if (!sess) {
 		ioSYSERR(0);
-		io_freeVar(v);
 		return NULL;
 	} else
 		memset(sess, 0, sizeof(struct tagSession));
@@ -38,19 +33,10 @@ initSession(int sock, ait_val_t * __restrict v)
 	else
 		sess->sess_retry = strtol(str, NULL, 0);
 
-	if (!(sess->sess_root = schedBegin())) {
-		ioLIBERR(sched);
-		free(sess);
-		io_freeVar(v);
-		return NULL;
-	}
-
 	sess->sess_buf = mqtt_msgAlloc(USHRT_MAX);
 	if (!sess->sess_buf) {
 		ioLIBERR(mqtt);
-		schedEnd(&sess->sess_root);
 		free(sess);
-		io_freeVar(v);
 		return NULL;
 	}
 
@@ -59,9 +45,7 @@ initSession(int sock, ait_val_t * __restrict v)
 	if (!sess->sess_srv) {
 		ioDEBUG(3, "Error:: in srv_Init #%d - %s", mqtt_GetErrno(), mqtt_GetError());
 		mqtt_msgFree(&sess->sess_buf, 42);
-		schedEnd(&sess->sess_root);
 		free(sess);
-		io_freeVar(v);
 		return NULL;
 	} else {
 		mqtt_srv_setCmd(sess->sess_srv, MQTT_TYPE_CONNECT, cmdCONNECT);
@@ -75,11 +59,10 @@ initSession(int sock, ait_val_t * __restrict v)
 
 	sess->sess_sock = sock;
 	strlcpy(sess->sess_addr, (char*) AIT_GET_STR(v), sizeof sess->sess_addr);
-	io_freeVar(v);
 	return sess;
 }
 
-static void
+void
 finiSession(struct tagSession *sess)
 {
 	struct tagStore *store;
@@ -93,7 +76,6 @@ finiSession(struct tagSession *sess)
 		call.FiniSessPUB(&cfg, pub, sess->sess_cid, sess->sess_user, "%");
 
 	SESS_ELEM_LOCK(sess);
-
 	while ((store = SLIST_FIRST(&sess->sess_subscr))) {
 		SLIST_REMOVE_HEAD(&sess->sess_subscr, st_node);
 
@@ -107,8 +89,6 @@ finiSession(struct tagSession *sess)
 	SESS_ELEM_UNLOCK(sess);
 	pthread_mutex_destroy(&sess->sess_mtx);
 
-	schedEnd(&sess->sess_root);
-
 	if (sess->sess_will.msg)
 		free(sess->sess_will.msg);
 	if (sess->sess_will.topic)
@@ -123,179 +103,81 @@ finiSession(struct tagSession *sess)
 	free(sess);
 }
 
-static void
-stopSession(struct tagSession *sess)
+static void *
+leaveClient(sched_task_t *task)
 {
-	mqtt_msg_t msg = { NULL, 0 };
+	struct tagSession *sess;
 	int ret;
 
 	ioTRACE(4);
 
+	assert(task);
+
+	sess = TASK_ARG(task);
 	assert(sess);
 
 	SESS_LOCK;
 	TAILQ_REMOVE(&Sessions, sess, sess_node);
 	SESS_UNLOCK;
 
-	ret = mqtt_msgDISCONNECT(&msg);
-	send(sess->sess_sock, msg.msg_base, ret, MSG_NOSIGNAL);
-	free(msg.msg_base);
+	ret = mqtt_msgDISCONNECT(sess->sess_buf);
+	send(TASK_FD(task), sess->sess_buf->msg_base, ret, MSG_NOSIGNAL);
 
-	ioDEBUG(1, "Close socket=%d", sess->sess_sock);
-	finiSession(sess);
-
+	ioDEBUG(1, "Close socket=%d", TASK_FD(task));
 	call.LOG(logg, "Session %s stopped from %s for user %s.\n", sess->sess_cid, 
 			sess->sess_addr, sess->sess_user);
+
+	finiSession(sess);
+	return NULL;
 }
 
-static int
-KASession(struct tagSession *sess)
+static void *
+dispatchSession(sched_task_t *task)
 {
-	mqtt_msg_t msg = { NULL, 0 };
-	int ret;
-	struct pollfd pfd;
+	int ret, len = 0;
+	struct tagSession *sess;
 
-	ioTRACE(4);
+	ioTRACE(2);
 
+	assert(task);
+
+	sess = TASK_ARG(task);
 	assert(sess);
 
-	/* ping request */
-	ret = mqtt_msgPINGREQ(&msg);
-	if ((ret = send(sess->sess_sock, msg.msg_base, ret, MSG_NOSIGNAL)) == -1) {
-		ioDEBUG(3, "Error:: send(%d) #%d - %s", sess->sess_sock, errno, strerror(errno));
-		return -1;
-	} else {
-		ioDEBUG(5, "Sended %d bytes for ping request", ret);
-		free(msg.msg_base);
-		memset(&msg, 0, sizeof msg);
-	}
-
-	pfd.fd = sess->sess_sock;
-	pfd.events = POLLIN | POLLPRI;
-	if ((ret = poll(&pfd, 1, sess->sess_ka * 1000)) == -1 || 
-			pfd.revents & (POLLERR | POLLHUP | POLLNVAL)) {
-		ioDEBUG(3, "Error:: poll(%d) #%d - %s", sess->sess_sock, errno, strerror(errno));
-		return -1;
-	} else if (!ret) {
-		ioDEBUG(5, "Warning:: Session is abandoned ... must be disconnect!");
-		return 1;
-	}
 	/* receive & decode packet */
-	if (recv(sess->sess_sock, sess->sess_buf->msg_base, sess->sess_buf->msg_len, 0) == -1) {
+	if ((ret = recv(TASK_FD(task), sess->sess_buf->msg_base, sess->sess_buf->msg_len, 0)) == -1) {
 		ioDEBUG(3, "Error:: recv(%d) #%d - %s", sess->sess_sock, errno, strerror(errno));
-		return -1;
+		finiSession(sess);
+		return NULL;
+	} else if (!ret) {
+		ioDEBUG(4, "Session %s EOF received.", sess->sess_cid);
+		finiSession(sess);
+		return NULL;
 	}
-	if (mqtt_readPINGRESP(sess->sess_buf)) {
-		ioDEBUG(5, "Warning:: Session is broken, not hear ping response ... must be disconnect!");
-		return 2;
-	}
 
-	/* Keep Alive is OK! */
-	return 0;
-}
-
-static void *
-thrSession(struct tagSession *sess)
-{
-	int ret, locKill = 42;
-	struct pollfd pfd;
-	struct mqtthdr *hdr;
-	ait_val_t *v;
-
-	pthread_cleanup_push((void(*)(void*)) stopSession, sess);
-	ioTRACE(2);
-
-	pfd.fd = sess->sess_sock;
-	pfd.events = POLLIN | POLLPRI;
-	while (!Kill && locKill) {
-		if ((ret = poll(&pfd, 1, sess->sess_ka * 1000)) == -1 || 
-				pfd.revents & (POLLERR | POLLHUP | POLLNVAL)) {
-			ioDEBUG(3, "Error:: poll(%d) #%d - %s", sess->sess_sock, errno, strerror(errno));
-			break;
-		} else if (!ret && (ret = KASession(sess))) {
-			call.LOG(logg, "Session %s keep-alive missing from %s for user %s ...\n", 
-					sess->sess_cid, sess->sess_addr, sess->sess_user);
-			break;
-		}
-		/* receive & decode packet */
-		if ((ret = recv(sess->sess_sock, sess->sess_buf->msg_base, sess->sess_buf->msg_len, 0)) == -1) {
-			ioDEBUG(3, "Error:: recv(%d) #%d - %s", sess->sess_sock, errno, strerror(errno));
-			break;
-		} else if (!ret) {
-			ioDEBUG(4, "Session %s EOF received.", sess->sess_cid);
-			break;
-		} else
-			hdr = (struct mqtthdr*) sess->sess_buf->msg_base;
-
+	do {
 		/* dispatch message type */
-		if (mqtt_srv_Dispatch(sess->sess_srv, sess))
+		if ((len = mqtt_srv_Dispatch(sess->sess_srv, ret, sess)) == -1) {
 			ioLIBERR(mqtt);
-		switch (hdr->mqtt_msg.type) {
-			case MQTT_TYPE_CONNECT:
-				ioDEBUG(5, "Exec CONNECT session");
-				if ((v = io_allocVar())) {
-					AIT_SET_STR(v, sess->sess_addr);
-					if (!schedEvent(root, startSession, v, (u_long) sess->sess_sock, sess, ret))
-						io_freeVar(v);
-				} else
-					ioLIBERR(mqtt);
+			ret = 0;
+		} else
+			ret -= len;
+	} while (len && ret > 0);
 
-				SESS_LOCK;
-				TAILQ_REMOVE(&Sessions, sess, sess_node);
-				SESS_UNLOCK;
-
-				locKill ^= locKill;
-				break;
-			case MQTT_TYPE_DISCONNECT:
-				ioDEBUG(5, "Exec DISCONNECT session");
-				finiSession(sess);
-
-				SESS_LOCK;
-				TAILQ_REMOVE(&Sessions, sess, sess_node);
-				SESS_UNLOCK;
-
-				locKill ^= locKill;
-				continue;
-			case MQTT_TYPE_PUBLISH:
-				ioDEBUG(5, "Exec PUBLISH topic QoS=%d", hdr->mqtt_msg.qos);
-				/*
-				if (cmdPUBLISH(sess))
-					locKill ^= locKill;
-					*/
-				break;
-			case MQTT_TYPE_PUBREL:
-				break;
-			case MQTT_TYPE_SUBSCRIBE:
-				break;
-			case MQTT_TYPE_UNSUBSCRIBE:
-				break;
-			case MQTT_TYPE_PINGREQ:
-				ioDEBUG(5, "Exec PINGREQ session");
-				break;
-			case MQTT_TYPE_PINGRESP:
-				ioDEBUG(5, "Exec PINGRESP session");
-				break;
-			default:
-				ioDEBUG(5, "Error:: Session %s, wrong command %d - DISCARDED", 
-						sess->sess_cid, hdr->mqtt_msg.type);
-				break;
-		}
-	}
-
-	pthread_cleanup_pop(locKill);
-	pthread_exit(NULL);
+	if (!schedRead(root, dispatchSession, TASK_ARG(task), TASK_FD(task), NULL, 0))
+		ioLIBERR(sched);
 	return NULL;
 }
 
-static void *
+void *
 startSession(sched_task_t *task)
 {
 	u_char basebuf[USHRT_MAX];
-	mqtt_msg_t msg = { NULL, 0 }, buf = { basebuf, sizeof basebuf };
+	mqtt_msg_t buf = { basebuf, sizeof basebuf };
 	mqtthdr_connflgs_t flg;
 	mqtthdr_connack_t cack;
 	struct tagSession *s, *sess = NULL;
-	int ret;
+	int ret, wlen;
 
 	ioTRACE(4);
 
@@ -304,8 +186,8 @@ startSession(sched_task_t *task)
 	if (!TASK_DATA(task)) {
 		/* flow from accept new clients */
 		sess = initSession(TASK_FD(task), TASK_ARG(task));
+		io_freeVar(TASK_ARG(task));
 		if (!sess) {
-			io_freeVar(TASK_ARG(task));
 			close(TASK_FD(task));
 			return NULL;
 		}
@@ -318,6 +200,8 @@ startSession(sched_task_t *task)
 		}
 	} else {
 		sess = TASK_DATA(task);
+		assert(sess);
+
 		buf.msg_len = TASK_DATLEN(task) > sizeof basebuf ? sizeof basebuf : TASK_DATLEN(task);
 		memcpy(buf.msg_base, sess->sess_buf->msg_base, buf.msg_len);
 	}
@@ -353,7 +237,7 @@ startSession(sched_task_t *task)
 		TAILQ_FOREACH(s, &Sessions, sess_node)
 			if (!strcmp(s->sess_cid, sess->sess_cid)) {
 				/* found stale session & disconnect it! */
-				stopSession(s);
+				schedWrite(root, leaveClient, sess, TASK_FD(task), NULL, 0);
 				break;
 			}
 	}
@@ -370,38 +254,27 @@ startSession(sched_task_t *task)
 		ret = MQTT_RETCODE_ACCEPTED;
 	}
 
-	ret = mqtt_msgCONNACK(&msg, ret);
-	if ((ret = send(sess->sess_sock, msg.msg_base, ret, 0)) == -1) {
-		ioDEBUG(3, "Error:: send(%d) #%d - %s", sess->sess_sock, errno, strerror(errno));
-		finiSession(sess);
-		return NULL;
-	} else {
-		ioDEBUG(5, "Sended %d bytes", ret);
-		free(msg.msg_base);
-		memset(&msg, 0, sizeof msg);
-	}
-
 	/* Start session thread OK ... */
 	SESS_LOCK;
 	TAILQ_INSERT_TAIL(&Sessions, sess, sess_node);
-	pthread_create(&sess->sess_tid, &attr, (void*(*)(void*)) thrSession, sess);
 	SESS_UNLOCK;
 
+	if (!schedRead(root, dispatchSession, sess, TASK_FD(task), NULL, 0))
+		ioLIBERR(sched);
+
 	call.LOG(logg, "Session %s started from %s for user %s (timeout=%d) OK!\n", sess->sess_cid, 
 			sess->sess_addr, sess->sess_user, sess->sess_ka);
-	return NULL;
 end:	/* close client connection */
-	ret = mqtt_msgCONNACK(&msg, ret);
-	if ((ret = send(sess->sess_sock, msg.msg_base, ret, 0)) == -1) {
-		ioDEBUG(3, "Error:: send(%d) #%d - %s", sess->sess_sock, errno, strerror(errno));
-	} else {
-		ioDEBUG(5, "Sended %d bytes", ret);
-		free(msg.msg_base);
-		memset(&msg, 0, sizeof msg);
-	}
+	wlen = mqtt_msgCONNACK(sess->sess_buf, ret);
+	if ((wlen = send(TASK_FD(task), sess->sess_buf->msg_base, wlen, 0)) == -1) {
+		ioDEBUG(3, "Error:: send(%d) #%d - %s", TASK_FD(task), errno, strerror(errno));
+	} else
+		ioDEBUG(5, "Sended %d bytes", wlen);
 
-	ioDEBUG(1, "Close client %s with socket=%d", sess->sess_addr, sess->sess_sock);
-	finiSession(sess);
+	if (ret != MQTT_RETCODE_ACCEPTED) {
+		ioDEBUG(1, "Close client %s with socket=%d", sess->sess_addr, TASK_FD(task));
+		finiSession(sess);
+	}
 	return NULL;
 }
 
@@ -421,7 +294,7 @@ acceptClient(sched_task_t *task)
 	if ((cli = accept(TASK_FD(task), &sa.sa, &sslen)) == -1)
 		goto end;
 	else
-		fcntl(TASK_FD(task), F_SETFL, fcntl(TASK_FD(task), F_GETFL, 0) | O_NONBLOCK);
+		fcntl(cli, F_SETFL, fcntl(cli, F_GETFL, 0) | O_NONBLOCK);
 
 	v = io_allocVar();
 	if (!v) {
@@ -431,6 +304,7 @@ acceptClient(sched_task_t *task)
 	} else {
 		memset(str, 0, sizeof str);
 		snprintf(str, sizeof str, "%s:%hu", io_n2addr(&sa, v), io_n2port(&sa));
+		AIT_FREE_VAL(v);
 		AIT_SET_STR(v, str);
 	}
 	ioDEBUG(1, "Connected client with socket=%d from %s", cli, AIT_GET_STR(v));
@@ -441,7 +315,7 @@ acceptClient(sched_task_t *task)
 		ioDEBUG(1, "Terminated client with socket=%d", cli);
 	}
 end:
-	if (!schedRead(TASK_ROOT(task), acceptClient, NULL, TASK_FD(task), NULL, 0))
+	if (!schedRead(root, acceptClient, NULL, TASK_FD(task), NULL, 0))
 		ioLIBERR(sched);
 	return NULL;
 }
@@ -468,13 +342,8 @@ Run(int sock)
 		return -1;
 	}
 
-	pthread_attr_init(&attr);
-	pthread_attr_setdetachstate(&attr, PTHREAD_CREATE_DETACHED);
-
 	schedPolling(root, &pl, NULL);
 	schedRun(root, &Kill);
-
-	pthread_attr_destroy(&attr);
 
 	/* free all undeleted elements into lists */
 	PUBS_LOCK;