1.已把空指针和防护都补上了：创建消息失败直接返回并删除 action；spooling/发送/序列化都会跳过空消息并记录错误，避免再因 m_pSen...

chenluhua1980

10 天以前 d2df50ab05a9d9a85a4b735786fe3f2b6e00186e

 SourceCode/Bond/Servo/HsmsAction.cpp

@@ -9,6 +9,7 @@
   m_nTimeout = 45;
   m_nResponseTime = 0;
   m_pContext = NULL;
   m_pSendMessage = NULL;
   m_hEvent = ::CreateEvent(NULL, TRUE, FALSE, NULL);
}

@@ -19,6 +20,7 @@
   m_nTimeout = nTimeout;
   m_nResponseTime = 0;
   m_pContext = NULL;
   m_pSendMessage = NULL;
   m_hEvent = ::CreateEvent(NULL, TRUE, FALSE, NULL);
}

@@ -41,6 +43,10 @@
   m_bNeedWaitReply = FALSE;
   m_nTimeout = 45;
   m_nResponseTime = 0;
   if (m_pSendMessage != NULL) {
      HSMS_Destroy1Message(m_pSendMessage);
      m_pSendMessage = NULL;
   }
   ::ResetEvent(m_hEvent);
}

@@ -73,6 +79,9 @@

void CHsmsAction::setSendMessage(IMessage* pMessage)
{
   if (m_pSendMessage != NULL && m_pSendMessage != pMessage) {
      HSMS_Destroy1Message(m_pSendMessage);
   }
   m_pSendMessage = pMessage;
}

@@ -110,12 +119,15 @@
int CHsmsAction::serialize(char* pszBuffer, int nBufferSize)
{
   int index = 0;
   if (m_pSendMessage == NULL) {
      return 0;
   }
   if (pszBuffer == nullptr) {
      index += sizeof(int);
      index += sizeof(m_nTimeout);
      index += sizeof(int);
      index += sizeof(BOOL);
      index += m_pSendMessage->serialize(pszBuffer, nBufferSize);
      index += m_pSendMessage->serialize(nullptr, 0);

      return index;
   }
@@ -157,7 +169,13 @@
   memcpy(&m_bNeedWaitReply, &pszBuffer[index], sizeof(BOOL));
   index += sizeof(BOOL);

   HSMS_Create1Message(m_pSendMessage, 1, 1 | REPLY, 1, 1);
   if (m_pSendMessage != NULL) {
      HSMS_Destroy1Message(m_pSendMessage);
      m_pSendMessage = NULL;
   }
   if (HSMS_Create1Message(m_pSendMessage, 1, 1 | REPLY, 1, 1) != 0 || m_pSendMessage == NULL) {
      return -1;
   }
   int nRet = m_pSendMessage->unserialize(&pszBuffer[index], nBufferSize - index);
   if (nRet < 0) return nRet;