Follow-up from "Артем Зенкевич P32081"
The following discussions from !117 (closed) should be addressed:
-
@av_ started a discussion:
Тут не рассматривается ситуация, когда
block->next == block, что в данном случае приведёт к бесконечному циклу. Имеет смысл добавить проверку и в таком случае вернуть результат с типом BSR_CORRUPTED. -
@av_ started a discussion:
В случае, если
last == NULL, вызов block_after(last) приведёт к ошибке. -
@av_ started a discussion:
- Тут надо бы ещё проверить, что выделенный блок не свободен и имеет нужный размер.
- Не все ресурсы, выделенные для теста, освобождены (так как в инициализацию кучи зашит вызов mmap, необходимо в конце вызвать munmap).
-
@av_ started a discussion:
- Неинформативное сообщение о результатах теста, хорошо бы добавить проверки после освобождения (освободился ли нужный блок, не освободились ли при этом случаййно занятые блоки).
- Освобождение ресурсов (munmap)
-
@av_ started a discussion:
- Аналогично — нужно проверить как минимум то, что выделенный блок действительно нужного размера.
- munmap
-
@av_ started a discussion:
?? Что такое x и для чего оно используется в данном тесте?
-
@av_ started a discussion:
- Аналогично, можно добавить проверки для результата выделения памяти и в случае, если эти проверки не пройдены, вывести сообщение об ошибке.
- munmap
-
@ArtemZen started a discussion: (+1 comment)
@ andrew