Сообщение для буфера в C

Question

У меня есть проект колледжа, где нужно преобразовать int в буфер char. Мне нужно использовать memcpy, но когда я копирую значения, это не работает, потому что msg_buf все еще пуст. У меня есть некоторые ограничения: - Мне нужно использовать memcpy, потому что мой учитель проверит мой код как memcmp (msg_str, & opcode, 2) == 0).

Вот мой код:

int message_to_buffer(struct message_t *msg, char **msg_buf){ 
    int opcode = htons(msg->opcode); 
    int c_type = htons(msg->c_type); 
    int result; 
    int buffer = sizeof(opcode) + sizeof(c_type); 

    switch(msg->c_type){ 

     case CT_RESULT: 

      result = htonl(msg->content.result); 
      buffer += sizeof(result); 
      *msg_buf = (char*)malloc(sizeof(char) * 12); 

      if(msg_buf == NULL) 
       return -1; 
      memcpy(*msg_buf,&opcode,sizeof(opcode)); 

      break; 

    }; 

    return buffer; 
} 

Что здесь не так?

Answer 1

В частности, вам необходимо скопировать шорты как шорты, а не ints. sizeof (short)! = sizeof (int) (обычно, в зависимости от архитектуры):

int message_to_buffer(struct message_t *msg, char **msg_buf){ 
    short opcode = htons(msg->opcode); 
    short c_type = htons(msg->c_type); 
    int result; 
    char* buffer = NULL, *buf_start=NULL; 
    *msg_buf = NULL; 

    switch(msg->c_type){ 

     case CT_RESULT: 

      result = htonl(msg->content.result); 
      buffer = (char*)malloc(sizeof(char) * 12); 

      if (buffer == NULL) 
       return -1; 
      buf_start = buffer; 
      memcpy(buffer,&opcode,sizeof(opcode)); // sizeof(short) == 2; sizeof(int) == 4 
      buffer += sizeof(opcode); 
      memcpy(buffer,&c_type,sizeof(c_type)); // sizeof(short) == 2; sizeof(int) == 4 
      buffer += sizeof(c_type); 
      memcpy(buffer,&result, sizeof(result)); 
      buffer += sizeof(result); 
      *msg_buf = buf_start; 
      break; 

    }; 

    return buffer - buf_start; 
} 

Answer 2

Почему бы вам не использовать функцию itoa для преобразования int в char*? Поэтому вы заменяете memcpy функцией itoa.

Ссылка: http://www.cplusplus.com/reference/cstdlib/itoa/

[EDIT] Если ваш компилятор не поддерживает itoa, вы можете использовать вместо sprintf.

Answer 3

Я думаю, ваша проблема может заключаться в том, что вы вызываете htons() на int. htons() предназначен для использования со значениями типа short, поэтому вы можете потерять верхние 16 бит вашего кода msg-> opcode и msg-> c_type. Попробуйте вместо этого заменить htons() на htonl().

Кроме того, похоже, что вы выделяете 12-байтовый буфер с помощью malloc(), но только записываете в него 4 байта, оставляя последние 8 байтов его неинициализированными/неопределенными. Это намеренно?