for cpp websocket core dump (#828)

zhaomingwork

2023-08-10 30f0c7ff2941ab08edb8cb257eb6cef74be42ec7

 funasr/runtime/websocket/websocket-server-2pass.cpp

@@ -54,7 +54,6 @@
}



nlohmann::json handle_result(FUNASR_RESULT result) {



    websocketpp::lib::error_code ec;

    nlohmann::json jsonresult;

    jsonresult["text"]="";

@@ -78,9 +77,8 @@
void WebSocketServer::do_decoder(

    std::vector<char>& buffer, websocketpp::connection_hdl& hdl,

    nlohmann::json& msg, std::vector<std::vector<std::string>>& punc_cache,

    websocketpp::lib::mutex& thread_lock, bool& is_final,

    std::string wav_name, FUNASR_HANDLE& tpass_online_handle) {

 
    websocketpp::lib::mutex& thread_lock, bool& is_final, std::string wav_name,

    FUNASR_HANDLE& tpass_online_handle) {

  // lock for each connection

  scoped_lock guard(thread_lock);

  FUNASR_RESULT Result = nullptr;

@@ -101,25 +99,28 @@
  }

 

  try {

    // loop to send chunk_size 800*2 data to asr engine.   TODO: chunk_size need get from client 
    // loop to send chunk_size 800*2 data to asr engine.   TODO: chunk_size need

    // get from client

    while (buffer.size() >= 800 * 2) {

      std::vector<char> subvector = {buffer.begin(),

                                      buffer.begin() + 800 * 2};

      std::vector<char> subvector = {buffer.begin(), buffer.begin() + 800 * 2};

      buffer.erase(buffer.begin(), buffer.begin() + 800 * 2);



      try{

        Result =

            FunTpassInferBuffer(tpass_handle, tpass_online_handle,

                                subvector.data(), subvector.size(), punc_cache,

                                false, msg["audio_fs"], msg["wav_format"], (ASR_TYPE)asr_mode_);

      }catch (std::exception const &e)

      {

        if (tpass_online_handle) {

          Result = FunTpassInferBuffer(tpass_handle, tpass_online_handle,

                                       subvector.data(), subvector.size(),

                                       punc_cache, false, msg["audio_fs"],

                                       msg["wav_format"], (ASR_TYPE)asr_mode_);



        } else {

          return;

        }

      } catch (std::exception const& e) {

          LOG(ERROR)<<e.what();

      }

      if (Result) {

        websocketpp::lib::error_code ec;

        nlohmann::json jsonresult =

            handle_result(Result);

        nlohmann::json jsonresult = handle_result(Result);

        jsonresult["wav_name"] = wav_name;

        jsonresult["is_final"] = false;

        if(jsonresult["text"] != "") {

@@ -133,16 +134,18 @@
        }

        FunASRFreeResult(Result);

      }



    }

    if(is_final){



      try{

        if (tpass_online_handle) {

        Result = FunTpassInferBuffer(tpass_handle, tpass_online_handle,

                                      buffer.data(), buffer.size(), punc_cache,

                                      is_final, msg["audio_fs"], msg["wav_format"], (ASR_TYPE)asr_mode_);

      }catch (std::exception const &e)

      {

                                       is_final, msg["audio_fs"],

                                       msg["wav_format"], (ASR_TYPE)asr_mode_);

        } else {

          return;

        }

      } catch (std::exception const& e) {

          LOG(ERROR)<<e.what();

      }

      for(auto &vec:punc_cache){

@@ -150,8 +153,7 @@
      }

      if (Result) {

        websocketpp::lib::error_code ec;

        nlohmann::json jsonresult =

            handle_result(Result);

        nlohmann::json jsonresult = handle_result(Result);

        jsonresult["wav_name"] = wav_name;

        jsonresult["is_final"] = true;

        if (is_ssl) {

@@ -191,24 +193,36 @@
  // data_msg->tpass_online_handle = tpass_online_handle;

  data_map.emplace(hdl, data_msg);

  LOG(INFO) << "on_open, active connections: " << data_map.size();

 
}



void WebSocketServer::on_close(websocketpp::connection_hdl hdl) {

  scoped_lock guard(m_lock);

void remove_hdl(

    websocketpp::connection_hdl hdl,

    std::map<websocketpp::connection_hdl, std::shared_ptr<FUNASR_MESSAGE>,

             std::owner_less<websocketpp::connection_hdl>>& data_map) {

  // return;

  std::shared_ptr<FUNASR_MESSAGE> data_msg  = nullptr;

  auto it_data = data_map.find(hdl);

  if (it_data != data_map.end()) {

    data_msg  = it_data->second;

  }

  else

  {

  } else {

    return;

  }

  scoped_lock guard_decoder(*(data_msg->thread_lock));  //wait for do_decoder finished and avoid access freed tpass_online_handle 
  // scoped_lock guard_decoder(*(data_msg->thread_lock));  //wait for do_decoder

  // finished and avoid access freed tpass_online_handle

  unique_lock guard_decoder(*(data_msg->thread_lock));

  if (data_msg->tpass_online_handle) {

  LOG(INFO) << "----------------FunTpassOnlineUninit----------------------";

  FunTpassOnlineUninit(data_msg->tpass_online_handle);

    data_msg->tpass_online_handle = nullptr;

  }

  guard_decoder.unlock();

  delete data_msg->thread_lock;

  data_map.erase(hdl);  // remove data vector when  connection is closed

}



void WebSocketServer::on_close(websocketpp::connection_hdl hdl) {

  scoped_lock guard(m_lock);

  remove_hdl(hdl, data_map);

  LOG(INFO) << "on_close, active connections: "<< data_map.size();

}



@@ -234,7 +248,7 @@
    iter++;

  }

  for (auto hdl : to_remove) {

    data_map.erase(hdl);

    remove_hdl(hdl, data_map);

    LOG(INFO) << "remove one connection ";

  }

}

@@ -248,6 +262,9 @@
  auto it_data = data_map.find(hdl);

  if (it_data != data_map.end()) {

    msg_data = it_data->second;

  } else {

    lock.unlock();

    return;

  }



  std::shared_ptr<std::vector<char>> sample_data_p = msg_data->samples;

@@ -282,15 +299,17 @@
      }

      if (jsonresult.contains("chunk_size")){

        if(msg_data->tpass_online_handle == NULL){

          std::vector<int> chunk_size_vec = jsonresult["chunk_size"].get<std::vector<int>>();

          LOG(INFO) << "----------------FunTpassOnlineInit----------------------";

          std::vector<int> chunk_size_vec =

              jsonresult["chunk_size"].get<std::vector<int>>();

          LOG(INFO)

              << "----------------FunTpassOnlineInit----------------------";

          FUNASR_HANDLE tpass_online_handle =

              FunTpassOnlineInit(tpass_handle, chunk_size_vec);

          msg_data->tpass_online_handle = tpass_online_handle;

        }

      }

      LOG(INFO) << "jsonresult=" << jsonresult << ", msg_data->msg="

                << msg_data->msg;

      LOG(INFO) << "jsonresult=" << jsonresult

                << ", msg_data->msg=" << msg_data->msg;

      if (jsonresult["is_speaking"] == false ||

          jsonresult["is_finished"] == true) {

        LOG(INFO) << "client done";

@@ -315,7 +334,8 @@
      if (isonline) {

        sample_data_p->insert(sample_data_p->end(), pcm_data,

                              pcm_data + num_samples);

        int setpsize = 800 * 2;  // TODO, need get from client 
        int setpsize =

            800 * 2;  // TODO, need get from client

          // if sample_data size > setpsize, we post data to decode

        if (sample_data_p->size() > setpsize) {

            int chunksize = floor(sample_data_p->size() / setpsize);