В ядре Linux устранена следующая уязвимость:
net/rose: исправлено разыменование нулевого указателя в Rose_transmit_link при повторном подключении.
syzkaller сообщил об ошибке [1], а воспроизводитель доступен по адресу [2]. Сокеты ROSE используют четыре значения sk->sk_state: TCP_CLOSE, TCP_LISTEN,
TCP_SYN_SENT и TCP_ESTABLISHED. Rose_connect() уже отклоняет
вызывает TCP_ESTABLISHED (-EISCONN) и TCP_CLOSE с SS_CONNECTING
(-ECONNREFUSED), но не проверяет TCP_SYN_SENT. Когда Rose_connect() вызывается второй раз во время первого соединения
попытка все еще продолжается (TCP_SYN_SENT), она перезаписывает
роза->сосед через Rose_get_neigh().
Если это возвращает NULL, сокет
остается Rose->state == ROSE_STATE_1, но Rose->neighbour == NULL. Когда сокет впоследствии закрывается, Rose_release() видит
ROSE_STATE_1 и вызывает Rose_write_internal() ->
rose_transmit_link(skb, NULL), вызывающее разыменование нулевого указателя. Для каждого подключения (2) используется второй метод подключения (), пока соединение уже установлено.
прогресс должен вернуться -УЖЕ.
Добавьте этот недостающий чек для
TCP_SYN_SENT для завершения проверки состояния в Rose_connect().
[1] https://syzkaller.appspot.com/bug?extid=d00f90e0af54102fb271
[2] https://gist.github.com/mrpre/9e6779e0d13e2c66779b1653fef80516
Показать оригинальное описание (EN)
In the Linux kernel, the following vulnerability has been resolved: net/rose: fix NULL pointer dereference in rose_transmit_link on reconnect syzkaller reported a bug [1], and the reproducer is available at [2]. ROSE sockets use four sk->sk_state values: TCP_CLOSE, TCP_LISTEN, TCP_SYN_SENT, and TCP_ESTABLISHED. rose_connect() already rejects calls for TCP_ESTABLISHED (-EISCONN) and TCP_CLOSE with SS_CONNECTING (-ECONNREFUSED), but lacks a check for TCP_SYN_SENT. When rose_connect() is called a second time while the first connection attempt is still in progress (TCP_SYN_SENT), it overwrites rose->neighbour via rose_get_neigh(). If that returns NULL, the socket is left with rose->state == ROSE_STATE_1 but rose->neighbour == NULL. When the socket is subsequently closed, rose_release() sees ROSE_STATE_1 and calls rose_write_internal() -> rose_transmit_link(skb, NULL), causing a NULL pointer dereference. Per connect(2), a second connect() while a connection is already in progress should return -EALREADY. Add this missing check for TCP_SYN_SENT to complete the state validation in rose_connect(). [1] https://syzkaller.appspot.com/bug?extid=d00f90e0af54102fb271 [2] https://gist.github.com/mrpre/9e6779e0d13e2c66779b1653fef80516