Как я могу диагностировать мою проблему десериализации буфера в структуру? - PullRequest
Новая
       28

Как я могу диагностировать мою проблему десериализации буфера в структуру?

0 голосов
/ 24 января 2019

Я пытаюсь понять, как динамически распределять память для c-строк в этой структуре.Пожалуйста, перейдите к выводу под кодом с моими объяснениями.

Вот код, который я запускаю: 

 NN #define USAGE "[get|put|quit] filenamesource filenamedest"
...
 20 enum cmd_type get_type(char *cmd) {
 21     if (0 == strncmp("put", cmd, 3)) {
 22         return PUT;
 23     }
 24     else if (0 == strncmp("get", cmd, 3)) {
 25         return GET;
 26     }
 27     else if (0 == strncmp("quit", cmd, 3)) {
 28         return QUIT;
 29     }
 30     else {
 31         return INV;
 32     }
 33 }
...
  0 int _recv_cmd(char *cmd_buf) {
 34     char type[MAX_FILENAME_LEN] = { '\0' };
 33     char src[MAX_FILENAME_LEN] = { '\0' };
 32     char dest[MAX_FILENAME_LEN] = { '\0' };
 31     size_t size = 0;
 30     enum error err = 0;
 29     struct command *cmd = NULL;
 28
 27     printf("Process %d received serialized command '%s'\n", getpid(), cmd_buf);
 26
 25     cmd = malloc(sizeof cmd);
 24     if (NULL == cmd) {
 23         perror("malloc");
 22         return -1;
 21     }
 20     if (5 != sscanf(cmd_buf, " %s %s %s %zu %u ", type, src, dest, &size, &err)) {
 19         fprintf(stderr, "sscanf failed to scan input.\n");
 18         free(cmd);
 17         return -1;
 16     }
 15
 14     printf("%s, %s, %s, %zu, %u\n", type, src, dest, size, err);
 13
 12     cmd->type = get_type(type);
 11     cmd->src = strdup(src);
 10     cmd->dest = strdup(dest);
  9     cmd->fsz = size;
  8     cmd->err = err;
  7
  6     print_cmd(cmd);
  5
  4     free(cmd);
  3
  2     return 0;
  1 }
...
  0 char * _get_input(char *buf) {
  1     // get input
  2     if (NULL == fgets(buf, CMD_LIMIT, stdin)) {
  3         fprintf(stderr, "fgets failed.\n");
  4         return NULL;
  5     }
  6     // remove newline
  7     int newline_pos = strcspn(buf, "\n");
  8     buf[newline_pos] = '\0';
  9
 10     return buf;
 11 }
 12
 13 int main(int argc, char **argv) {
 14     char cmd_buf[CMD_LIMIT] = { '\0' };
 15
 16     while(1) {
 17
 18         printf("Enter command of the form '%s':\n", USAGE);
 19
 20         _recv_cmd(_get_input(cmd_buf));
 21     }
 22
 23     exit(EXIT_SUCCESS);
 24 }

Вот тип команды: 

 29 struct command {
 30     enum cmd_type type;
 31     char *src;
 32     char *dest;
 33     size_t fsz;
 34     enum error err;
 35 };

Это вывод, который я получаю при запуске программирования.Обратите внимание на пробелы, которые я добавляю между примерами. 

➜  assignment01 git:(master) ✗ ./test
Enter command of the form '[get|put|quit] filenamesource filenamedest':
put hi hi 25 0
Process 53001 received serialized command 'put hi hi 25 0'
put, hi, hi, 25, 0
type: 2, source:  , dest: , size: 25, err: 0
Enter command of the form '[get|put|quit] filenamesource filenamedest':
put  hi hi 25 0
Process 53022 received serialized command 'put  hi hi 25 0'
put, hi, hi, 25, 0
type: 2, source: 'p, dest: , size: 25, err: 0
Enter command of the form '[get|put|quit] filenamesource filenamedest':
get  what is 30 1
Process 53022 received serialized command 'get  what is 30 1'
get, what, is, 30, 1
type: 1, source: what, dest: is, size: 30, err: 1

Как видно из выходных данных, содержимое буфера выглядит точно так, как я и ожидал ("полученный сериализованный ...").Ничего необычного там нет.

Выходные данные в строках, которые находятся непосредственно под приглашением, выводятся из строки

printf("%s, %s, %s, %zu, %u\n", type, src, dest, size, err);,

, указывающей, что sscanf работал точно так, как я ожидал.Почему же тогда содержимое cmd так испорчено?

Я понимаю, что место для указателя на строки src и dest предоставляется с помощью malloc для cmd, и это strdupmalloc'ing место для (нулевой символ и все) для c-строк в самой структуре.Что дает?

Я уже пробовал альтернативные методы, такие как malloc'ing и затем strncpy или memmove.

Я просто добавлю, что эти ведущие подчеркивания выбираются так, чтобы имена функций не конфликтовали с этимив общемЭто минимальный рабочий пример, используемый для воссоздания проблемы, а не исходный код, с которым я работал.То же самое относится и к освобождению строк и т. Д. Просто упускается из виду при создании этого небольшого тестового приложения.

print_cmd просто распечатывает структуру с соответствующим форматированием, в точности, как можно было бы представить.

1 Ответ

0 голосов
/ 25 января 2019

Эта строка: 

cmd = malloc(sizeof cmd);

Выделяет достаточно места для указателя на структуру, но вы хотите достаточно места для самой структуры. Вы можете решить это с 

cmd = malloc(sizeof *cmd);

или 

cmd = malloc(sizeof(struct command));
...