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

В целом, любая функция, которая не проверяет границы в аргументы. Список был бы

• , добирается ()
• scanf ()
• strcpy ()
• strcat ()

, необходимо использовать ограниченные версии размера как stncpy, strncat, fgets, и т.д. Затем быть осторожными при предоставлении предела размера; учтите '\0' завершений строки.

кроме того, массивы НЕ связываются зарегистрированные C или C++. Следующий пример вызвал бы ошибки. См. прочь одной ошибкой

int foo[3];
foo[3] = WALKED_OFF_END_OF_ARRAY;

редактирование : Скопированные ответы @MrValdez, @Denton Дворянство

Valgrind является Вашим новым лучшим другом.

valgrind - tool=memcheck - leak-check=full./a.out

Вопрос запускается в неправильном конце, я боюсь. Это предполагает, что переполнение буфера происходит в других функциях. Наиболее распространенной причиной является оператор ++, по моему опыту, или альтернативно отсутствие оператора! =.

лучшим решением найти пакет тех является / GS в Visual Studio 2005/8. Это не найдет всех их, но это - дешевый способ уменьшить количество необходимого физического труда.

Вот некоторые функции, что я нашел, что опасны:

• добирается () - Это не проверяет длину переменной и может перезаписать память, если вход больше, чем буфер переменной.
• scanf () - я так рад, что Visual Studio сказала мне, что эта функция удерживается от использования. Это было легкой фиксацией.
• strcpy () - Если пространство памяти источника больше, чем место назначения, данные после того, как место назначения перезаписывается.

К сожалению, любой массив может привести к переполнению буфера:

uint32_t foo[3];
foo[3] = WALKED_OFF_END_OF_ARRAY;

С точки зрения функций, sprintf счастливо уйдет от конца буфера. Это может быть заменено snprintf.

Memcpy () является другим опасный.

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

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

Какую версию Visual Studio Вы используете? В 2008 со всеми включенными предупреждениями, все функции Вы упоминаете (и больше) предупреждают Вас, что они удерживаются от использования.

, Возможно, Вы могли проверить, что все предупреждения включены и позволяют компилятору сделать тяжелую работу для Вас?

Как примечание стороны, превосходное безопасный код записи делает отличную работу, объясняя различное ловушки некоторых более старых функций.

У меня есть несколько та же проблема на кодовой базе, я продолжаю работать. Мой совет: опасайтесь любых функций C, которые похожи на str* () и мадам* (). Также опасайтесь чего-либо, что берет указатель на буфер без длины. Так как кажется, что у Вас есть шанс использовать C++, я был бы в самых вопиющих случаях пытаться использовать контейнеры C++ для вещей: вектор, строка, карта, и т.д. Они делают Вашу жизнь намного легче.

кроме того, автоматизированные проблемные инструменты обнаружения замечательны иметь. Если бы можно использовать valgrind, я рекомендовал бы его. Также Рациональный Очищают, чрезвычайно мощно, хотя не дешевый.

Следующая ссылка должна дать Вам всесторонний взгляд на функции защиты в C++ (которые снабжаются постфиксом '_s' для решения проблем как переполнение): http://msdn.microsoft.com/en-us/library/8ef0s5kh (По сравнению с 80) РЕДАКТИРОВАНИЕМ .aspx

: Эта ссылка содержит определенные функции, которые были заменены: http://msdn.microsoft.com/en-us/library/wd3wzwts (По сравнению с 80) РЕДАКТИРОВАНИЕМ .aspx

: я должен упомянуть, что это методы Microsoft, но ссылка все еще полезна для идентификации функций, которые считали красным флагом.

В основном, что-либо, что принимает указатель и записи к нему, не проверяя длину. Так вещь как strcpy (), sprintf () и т.д.

Еще одна проблема в C - это функция strncpy (). Многие люди не понимают, что можно бесплатно возвращать строку, не оканчивающуюся нулем.