socket 发送指针数据的问题

jsjqzhang

定义了结构如下:
#pragma pack(1)
struct __CollectorCommCmd
{
    uint32_t code;
    uint32_t len;   
    char * pData;
  
    __CollectorCommCmd()
    {
    pData = NULL;
    }

    ~__CollectorCommCmd()
    {
        if(pData != NULL)
        {
            delete pData;
        }
    }
}
;
#pragma pack()写了如下函数
HRESULT CCollector::sendCmdToIB(__CollectorCommCmd * i_p_cmd)
{
    HRESULT ret;   
    {
        int  cmd_len = sizeof(i_p_cmd->code) +sizeof(i_p_cmd->len) + i_p_cmd->len;
        char ** buff ;
        *buff = new char[cmd_len];
        memcpy(*buff, i_p_cmd, (sizeof(i_p_cmd->code) + sizeof(i_p_cmd->len)));
        //问题可能在这里, 解析到此处会访问非法指针
        memcpy(*buff +8,i_p_cmd->pData),i_p_cmd->len);
        ret = collector_comm->sendToCommB(*buff, cmd_len);
        delete [] buff;
    }   
    return ret;
}

HRESULT CCollectorComm::sendToCommB(const char * i_p_buff, const int & i_r_len)
{
    HRESULT ret = sendToCommForce(b_sockfd, i_p_buff, i_r_len);
    return ret;
}


HRESULT SockComm::sendToCommForce(const int & i_r_sockfd, const char * i_p_buff, const int & i_r_len)
{
    ......//封装具体的 socket 的write
    return S_OK;
}

严重怀疑是sendCmdToIB函数有问题,所以作了以下写法

HRESULT CCollector::sendCmdToIB(__CollectorCommCmd * i_p_cmd)
{
    HRESULT ret;   
    {
        int  cmd_len = sizeof(i_p_cmd->code) +sizeof(i_p_cmd->len) + i_p_cmd->len;
        
        char  *buff = new char[cmd_len];
        memcpy(buff, i_p_cmd, (sizeof(i_p_cmd->code) + sizeof(i_p_cmd->len)));
        //问题可能在这里
        memcpy(buff +8,i_p_cmd->pData),i_p_cmd->len);
        ret = collector_comm->sendToCommB(buff, cmd_len);
        delete [] buff;
    }   
    return ret;
}
//这个写法能成功传送数据,命令头和命令长度正确,但是数据乱码,
希望高手能帮忙,在包结构不改变的情况下,帮忙修改一下

封装数据的操作 如下

HRESULT FUNC(__PerfTaskItem * temp_perf_task_item)
{
   HRESULT ret;
   __CollectorCommCmd  cmd;
   cmd.code  = CMD_OUT_PERF_TASK;//#define
   cmd.len = sizeof(__PerfTaskItem);
   cmd.pData= (char *)malloc(sizeof(__PerfTaskItem)));
   memcpy(cmd.pData,temp_perf_task_item,sizeof(__PerfTaskItem));
   ret =sendCmdToIB(&cmd);
   return  ret;
}

知道了数据结构的长度,根据数据结构长度来分配内存

selfrun

buff没初始化

lanneret_sky

memcpy(buff +8,i_p_cmd->pData),i_p_cmd->len);
溢出了，复制长度比buff大
char  *buff = new char[cmd_len＋8];
这样试试

jsjqzhang

经过测试,发现在发送方先使用了
memcpy(buff,i_p_cmd,(sizeof(i_p_cmd->code)+sizeof(i_p_cmd->len)));

copy 了整形数据, 能解析成功, 但是
memcpy(buff+8,i_p_cmd->pData,i_p_cmd->len); 就开始出现乱码,

直拉打出i_p_cmd->pData 数据又是正常的