Diff for /mqtt/src/daemon.c between versions 1.2.2.24 and 1.2.2.26

version 1.2.2.24, 2012/05/05 13:39:27 version 1.2.2.26, 2012/05/05 15:04:29
Line 23  initSession(int sock, ait_val_t * __restrict v) Line 23  initSession(int sock, ait_val_t * __restrict v)
         } else          } else
                 memset(sess, 0, sizeof(struct tagSession));                  memset(sess, 0, sizeof(struct tagSession));
   
         pthread_mutex_init(&sess->sess_mtx, NULL);  
   
         SLIST_INIT(&sess->sess_subscr);          SLIST_INIT(&sess->sess_subscr);
   
         str = cfg_getAttribute(&cfg, "mqttd", "retry");          str = cfg_getAttribute(&cfg, "mqttd", "retry");
Line 62  initSession(int sock, ait_val_t * __restrict v) Line 60  initSession(int sock, ait_val_t * __restrict v)
         return sess;          return sess;
 }  }
   
voidstatic void
 finiSession(struct tagSession *sess)  finiSession(struct tagSession *sess)
 {  {
         struct tagStore *store;          struct tagStore *store;
Line 75  finiSession(struct tagSession *sess) Line 73  finiSession(struct tagSession *sess)
         if (call.FiniSessPUB)          if (call.FiniSessPUB)
                 call.FiniSessPUB(&cfg, pub, sess->sess_cid, sess->sess_user, "%");                  call.FiniSessPUB(&cfg, pub, sess->sess_cid, sess->sess_user, "%");
   
         SESS_ELEM_LOCK(sess);  
         while ((store = SLIST_FIRST(&sess->sess_subscr))) {          while ((store = SLIST_FIRST(&sess->sess_subscr))) {
                 SLIST_REMOVE_HEAD(&sess->sess_subscr, st_node);                  SLIST_REMOVE_HEAD(&sess->sess_subscr, st_node);
   
Line 86  finiSession(struct tagSession *sess) Line 83  finiSession(struct tagSession *sess)
   
                 free(store);                  free(store);
         }          }
         SESS_ELEM_UNLOCK(sess);  
         pthread_mutex_destroy(&sess->sess_mtx);  
   
         if (sess->sess_will.msg)          if (sess->sess_will.msg)
                 free(sess->sess_will.msg);                  free(sess->sess_will.msg);
Line 116  leaveClient(sched_task_t *task) Line 111  leaveClient(sched_task_t *task)
         sess = TASK_ARG(task);          sess = TASK_ARG(task);
         assert(sess);          assert(sess);
   
         SESS_LOCK;  
         TAILQ_REMOVE(&Sessions, sess, sess_node);          TAILQ_REMOVE(&Sessions, sess, sess_node);
         SESS_UNLOCK;  
   
         ret = mqtt_msgDISCONNECT(sess->sess_buf);          ret = mqtt_msgDISCONNECT(sess->sess_buf);
         send(TASK_FD(task), sess->sess_buf->msg_base, ret, MSG_NOSIGNAL);          send(TASK_FD(task), sess->sess_buf->msg_base, ret, MSG_NOSIGNAL);
Line 147  dispatchSession(sched_task_t *task) Line 140  dispatchSession(sched_task_t *task)
         /* receive & decode packet */          /* receive & decode packet */
         if ((ret = recv(TASK_FD(task), 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));                  ioDEBUG(3, "Error:: recv(%d) #%d - %s", sess->sess_sock, errno, strerror(errno));
                   TAILQ_REMOVE(&Sessions, sess, sess_node);
                 finiSession(sess);                  finiSession(sess);
                 return NULL;                  return NULL;
         } else if (!ret) {          } else if (!ret) {
                 ioDEBUG(4, "Session %s EOF received.", sess->sess_cid);                  ioDEBUG(4, "Session %s EOF received.", sess->sess_cid);
                   TAILQ_REMOVE(&Sessions, sess, sess_node);
                 finiSession(sess);                  finiSession(sess);
                 return NULL;                  return NULL;
         }          }
   
         do {          do {
                 /* dispatch message type */                  /* dispatch message type */
                if ((len = mqtt_srv_Dispatch(sess->sess_srv, ret, sess)) == -1) {                if ((len = mqtt_srv_Dispatch(sess->sess_srv, ret, sess)) < 0) {
                        ioLIBERR(mqtt);                        if (len == -1)
                        ret = 0;                                ioLIBERR(mqtt);
                         TAILQ_REMOVE(&Sessions, sess, sess_node);
                         finiSession(sess);
                 } else                  } else
                         ret -= len;                          ret -= len;
        } while (len && ret > 0);        } while (len > 0 && ret > 0);
   
        if (!schedRead(root, dispatchSession, TASK_ARG(task), TASK_FD(task), NULL, 0))        if (len >= 0 && !schedRead(root, dispatchSession, TASK_ARG(task), TASK_FD(task), NULL, 0)) {
                 ioLIBERR(sched);                  ioLIBERR(sched);
                   TAILQ_REMOVE(&Sessions, sess, sess_node);
                   finiSession(sess);
           }
         return NULL;          return NULL;
 }  }
   
Line 254  startSession(sched_task_t *task) Line 254  startSession(sched_task_t *task)
                 ret = MQTT_RETCODE_ACCEPTED;                  ret = MQTT_RETCODE_ACCEPTED;
         }          }
   
        /* Start session thread OK ... */        /* Start session task OK ... */
        SESS_LOCK;        if (!schedRead(root, dispatchSession, sess, TASK_FD(task), NULL, 0)) {
        TAILQ_INSERT_TAIL(&Sessions, sess, sess_node); 
        SESS_UNLOCK; 
 
        if (!schedRead(root, dispatchSession, sess, TASK_FD(task), NULL, 0)) 
                 ioLIBERR(sched);                  ioLIBERR(sched);
                   ret = MQTT_RETCODE_DENIED;
           } else
                   TAILQ_INSERT_TAIL(&Sessions, sess, sess_node);
   
         call.LOG(logg, "Session %s started from %s for user %s (timeout=%d) OK!\n", sess->sess_cid,           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);                          sess->sess_addr, sess->sess_user, sess->sess_ka);
Line 326  int Line 325  int
 Run(int sock)  Run(int sock)
 {  {
         struct tagPub *pub;          struct tagPub *pub;
           struct tagSession *sess;
         struct timespec pl = { 0, 100000000 };          struct timespec pl = { 0, 100000000 };
   
         ioTRACE(1);          ioTRACE(1);
Line 346  Run(int sock) Line 346  Run(int sock)
         schedRun(root, &Kill);          schedRun(root, &Kill);
   
         /* free all undeleted elements into lists */          /* free all undeleted elements into lists */
         PUBS_LOCK;  
         TAILQ_FOREACH(pub, &Pubs, pub_node) {          TAILQ_FOREACH(pub, &Pubs, pub_node) {
                 TAILQ_REMOVE(&Pubs, pub, pub_node);                  TAILQ_REMOVE(&Pubs, pub, pub_node);
   
Line 354  Run(int sock) Line 353  Run(int sock)
                 if (pub->pub_packet.msg_base)                  if (pub->pub_packet.msg_base)
                         free(pub->pub_packet.msg_base);                          free(pub->pub_packet.msg_base);
         }          }
        PUBS_UNLOCK;        TAILQ_FOREACH(sess, &Sessions, sess_node) {
                 TAILQ_REMOVE(&Sessions, sess, sess_node);
 
                 finiSession(sess);
         }
         return 0;          return 0;
 }  }

Removed from v.1.2.2.24  
changed lines
  Added in v.1.2.2.26


FreeBSD-CVSweb <freebsd-cvsweb@FreeBSD.org>