переполнение буфера malloc (C)

Question

Приносим извинения, если об этом ответили раньше, но я сам сводил с ума поиски ответа. Я использую таНос для создания массива:

char *message = (char*) malloc(31 *sizeof(char)); 

Чтобы заставить пользователя ввести сообщение из максимум 30 символов:

fgets(message, 30, stdin); 

В другой функции я потом попытался преобразовать сообщение, чтобы понизить и сохраните его в другой строке (надеюсь, равной длины, как сообщение).

int length_message = strlen(message) - 1; 

char *temp1 = (char*) malloc(length_message * sizeof(char)); 

for(int i=0; message[i] != '\0'; i++) 
{ 
     temp1[i] = tolower(message[i]);  
} 

Но когда я пришел, чтобы отобразить новую строку

puts(temp1); 

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

Кроме того при проверке длины моего TEMP1 массива и значения я создал для длины сообщения, используя:

printf("length of temp array: %d", strlen(temp1)); 
printf("length of message: %d\n", length_message); 

Который вернул нужный length_message (количество вводимых символов пользователя в сообщении), но неправильный номер для длины моего массива temp! Я неправильно присвоил память для моего временного массива или неправильно написал ли я этому массиву?

Из чего я прочитал другие темы, у меня, похоже, есть ошибка переполнения буфера? Как я могу держать это под контролем?

Заранее благодарен!

Answer 1

Есть пара проблем с вашим кодом.

Фактическая ошибка вызвана тем, что вы не копируете завершающий символ 0 из исходной строки в пункт назначения. Вам нужно написать temp[i] = 0; после цикла for.

Кроме того, у вас действительно есть возможность переполнения буфера, так как вы являетесь malloc(), при этом один символ меньше для копирования, чем длина строки. (Почему ...? Вы должны выделить один более ...)

Другие проблемы включают в себя ужасныеcasting of void *, избыточные (и, следовательно, небезопасные) sizeof(char) и жесткий кодировкой буфер размер.

Вместо

char *message = (char*) malloc(31 *sizeof(char)); 

попробовать

#define BUF_SIZE 31 

char *message = malloc(BUF_SIZE); 
fgets(message, BUF_SIZE, stdin); 

или если вы чувствуете педантичным, то

char *message = malloc(BUF_SIZE * sizeof message[0]); 

но никогда использование жестко закодированных типа в sizeof(), когда вы malloc() в памяти. Но в любом случае, я не думаю, что вам нужно malloc(). Почему бы вам не использовать автоматический массив («выделенный стек», хотя это неправильное имя)? Тогда вы можете воспользоваться оператором sizeof при вводе пользователя.

Кроме того, strlen() и несколько других функций stdlib возвращаются size_t. Вы должны использовать этот тип, а не int.

Answer 2

Это:

strlen(message) - 1 

, конечно, должен быть таким:

strlen(message) + 1 

Вы сделали опечатку.