search

Etersoft / libnss-role

NSS API library and admin tools for roles and privilegies
GNU Lesser General Public License v2.1
1 stars 6 forks source link

fix freing (is found by cppcheck) #7

Open vitlav opened 4 years ago

ghost commented 4 years ago

Добрый день, коллеги. У меня есть небольшая просьба: нам бы надо покрывать функции, которые мы ремонтируем, unit-тестами. Шаблоны тестов уже можно подсмотреть в директории test.

vitlav commented 4 years ago

Если вы знаете, как покрыть тестом вызов free с неправильным адресом, то покажите пример такого теста. Мне кажется, это невозможно.

ghost commented 4 years ago

Если вы знаете, как покрыть тестом вызов free с неправильным адресом, то покажите пример такого теста. Мне кажется, это невозможно.

Добрый день, @vitlav . Покрывать надо не вызов библиотечной функции free() , а всю функцию, которую чиним.

vitlav commented 4 years ago

Если вы знаете, как покрыть тестом вызов free с неправильным адресом, то покажите пример такого теста. Мне кажется, это невозможно.

Добрый день, @vitlav . Покрывать надо не вызов библиотечной функции free() , а всю функцию, которую чиним.

В ближайшее время не смогу заняться тестами.

ghost commented 4 years ago

В результате code review было выяснено, что логика кода сломана: Вызов free() в librole_realloc_buffer() нарушает владение указателем. Я накидаю сверху коммитами ремонт логики кода.

Также обнаружил неиспользуемую функцию librole_en_vector() которую надо удалить.

vitlav commented 3 years ago

В результате code review было выяснено, что логика кода сломана: Вызов free() в librole_realloc_buffer() нарушает владение указателем. Я накидаю сверху коммитами ремонт логики кода.

Также обнаружил неиспользуемую функцию librole_en_vector() которую надо удалить.

Чем закончилось?

ghost commented 3 years ago

@mastersin , what part of code you are talking about?