> При том что вся увизгвимость как раз крутилась вокруг факта, что node
> таки иногда оказывалось wild pointer где не ждали.Если кто на бронепоезде, assert дебажная фича, ловящая programming error - оно выносит процесс с аварийным завершением. Это не является штатным состоянием дел, последний писк для разработчика "ты лох и наломал дров!". Юзеру пищать бесполезно, он выругается и перезапустит.
> wild pointer где не ждали.
Так это и есть баг - сам факт что он wild, в комите описано как это получается. Предлагается законопатить и сделать вид что wild-а не было? :) Там фикс кроме всего прочего меняет еще и работу c этим, tr_new() добавили. После чего оно уже не "free", так что use after free там уже быть не должно.
> Угадайте, что у нас такое TR_ASSERT?
В дебажном варианте там нормальный ассерт. Но это имеет смысл для разработчиков способных на такое внятно реагировать. Чего юзер с явным programming error должен сделать?
> P.S. а что, с шитхаба теперь банально скопировать патч нельзя, кнопка raw
> куда-то исчезла?
Без понятия, в git log работает.