--- mqtt/src/daemon.c	2011/12/16 12:16:17	1.1.2.24
+++ mqtt/src/daemon.c	2011/12/16 13:34:41	1.1.2.28
@@ -79,6 +79,10 @@ stopSession(struct tagSession *sess)
 	mqtt_msg_t msg = { NULL, 0 };
 	int ret;
 
+	ioTRACE(4);
+
+	assert(sess);
+
 	pthread_mutex_lock(&mtx_sess);
 	TAILQ_REMOVE(&Sessions, sess, sess_node);
 	pthread_mutex_unlock(&mtx_sess);
@@ -89,20 +93,105 @@ stopSession(struct tagSession *sess)
 	else {
 		ioDEBUG(5, "Sended %d bytes for disconnect", ret);
 		free(msg.msg_base);
-		msg.msg_len = 0;
+		memset(&msg, 0, sizeof msg);
 	}
 
 	ioDEBUG(1, "Close socket=%d", sess->sess_sock);
 	finiSession(sess);
+
+	call.LOG(logg, "Session %s stopped from %s for user %s.\n", sess->sess_cid, 
+			sess->sess_addr, sess->sess_user);
 }
 
+static int
+KASession(struct tagSession *sess)
+{
+	u_char basebuf[USHRT_MAX];
+	mqtt_msg_t msg = { NULL, 0 }, buf = { basebuf, sizeof basebuf };
+	int ret;
+	struct pollfd pfd;
+
+	ioTRACE(4);
+
+	assert(sess);
+
+	/* ping request */
+	ret = mqtt_msgPINGREQ(&msg);
+	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));
+		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)) == -1 || 
+			pfd.revents & (POLLERR | POLLHUP | POLLNVAL)) {
+		ioDEBUG(3, "Error:: poll(%d) #%d - %s", sess->sess_sock, errno, strerror(errno));
+		return -1;
+	} else if (!ret) {
+		/* problem!!! session is abandoned ... must be disconnect! */
+		return 1;
+	}
+	/* receive & decode packet */
+	if (recv(sess->sess_sock, buf.msg_base, buf.msg_len, 0) == -1) {
+		ioDEBUG(3, "Error:: recv(%d) #%d - %s", sess->sess_sock, errno, strerror(errno));
+		return -1;
+	}
+	if (mqtt_readPINGRESP(&buf)) {
+		/* problem!!! session is broken, not hear ping response ... must be disconnect! */
+		return 2;
+	}
+
+	/* Keep Alive is OK! */
+	return 0;
+}
+
 static void *
 thrSession(struct tagSession *sess)
 {
-	pthread_cleanup_push((void(*)(void*)) stopSession, sess);
+	u_char basebuf[USHRT_MAX];
+	mqtt_msg_t msg = { NULL, 0 }, buf = { basebuf, sizeof basebuf };
+	int ret;
+	struct pollfd pfd;
+	struct mqtthdr *hdr;
 
+	pthread_cleanup_push((void(*)(void*)) stopSession, sess);
 	ioTRACE(2);
 
+	pfd.fd = sess->sess_sock;
+	pfd.events = POLLIN | POLLPRI;
+	while (!Kill) {
+		if ((ret = poll(&pfd, 1, sess->sess_ka)) == -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, buf.msg_base, 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*) buf.msg_base;
+
+		switch (hdr->mqtt_msg.type) {
+			default:
+				ioDEBUG(5, "Error:: Session %s, wrong command %d - DISCARDED", 
+						sess->sess_cid, hdr->mqtt_msg.type);
+				break;
+		}
+	}
+
 	pthread_cleanup_pop(42);
 	pthread_exit(NULL);
 }
@@ -120,6 +209,8 @@ startSession(sched_task_t *task)
 
 	ioTRACE(4);
 
+	assert(task);
+
 	sess = initSession(TASK_FD(task), TASK_ARG(task));
 	if (!sess) {
 		io_freeVar(TASK_ARG(task));
@@ -162,6 +253,7 @@ startSession(sched_task_t *task)
 		ioDEBUG(2, "Old session %s should be disconnect!", sess->sess_cid);
 		TAILQ_FOREACH(s, &Sessions, sess_node)
 			if (!strcmp(s->sess_cid, sess->sess_cid)) {
+				/* found stale session & disconnect it! */
 				stopSession(s);
 				break;
 			}
@@ -186,7 +278,7 @@ startSession(sched_task_t *task)
 	} else {
 		ioDEBUG(5, "Sended %d bytes", ret);
 		free(msg.msg_base);
-		msg.msg_len = 0;
+		memset(&msg, 0, sizeof msg);
 	}
 
 	/* Start session thread OK ... */
@@ -210,7 +302,7 @@ end:	/* close client connection */
 	} else {
 		ioDEBUG(5, "Sended %d bytes", ret);
 		free(msg.msg_base);
-		msg.msg_len = 0;
+		memset(&msg, 0, sizeof msg);
 	}
 
 	ioDEBUG(1, "Close client %s with socket=%d", sess->sess_addr, sess->sess_sock);