# WW 5

[Дмитрий Свиридкин]

src/point_list.c:66:14: warning: passing argument 2 of ‘apply’ from incompatible pointer type [-Wincompatible-pointer-types]

apply(list, print_point, out);

Вот так делать нельзя. Даже если собирается. Кастить указатели на функции с разными сигнатурами -- прямой путь к UB

Течет память

count?

3/10

[Дмитрий Свиридкин]

0.
src/main.c:27:23: warning: implicit declaration of function ‘get_point’; did you mean ‘add_point’? [-Wimplicit-function-declaration]

point_node *p_node = get_point(node);

У вас сразу должно было все упасть из-за UB с невалидными указателями.

1.
Преобразование считанных чисел из бинарного файла некорректное. char часто бывает знаковым.

2. количество аргументов, переданных в программу по-хорошему надо проверять.

3. Используйте типы с фиксированным размером при работе с бинарными данными (uint8_t, int32_t и т.д.)

4. Перепутан порядок байт.

---

4/10

[zhemchuzhina.elizaveta]

Знаковый char на доп. баллы.

[Дмитрий Свиридкин]

Знаковый char не про доп. баллы: Если в файле будут байты [255 255 0], char, в который вы читаете, станет отрицательным. int, в который сохраняете, станет отрицательным, а после сдвига произойдет UB.

Собственно, числа, у которых хоть один байт больше 127, у вас чистаются неверно.

---

7.5