Это-то понятно, в реальности эффект от него заметить нельзя. Но сам факт наличия подобных ошибок говорит многое о качестве кода. Почему половину полей-указателей в структуре они освобождают при ее удалении, а вторую — нет? Если бы не вызывали free совсем — было бы понятно, типа такие стремные code guidelines; а так — ни туда, ни сюда.
При монтировании тома с файловой системой udfs для него выделяется структура (контекст), одним из полей которой является указатель на кусок памяти, используемый для кеширования разных критичных секторов. После отмонтирования контекст освобождается, все остальные структуры кроме этого куска тоже освобождаются. В чем смысл оставлять его, если заведомо нигде больше ссылок туда быть не может?
17 comments
recommend
bookmark
subscribe
Ну, ну. Не столько он работает, чтобы это было страшно.
Это-то понятно, в реальности эффект от него заметить нельзя. Но сам факт наличия подобных ошибок говорит многое о качестве кода. Почему половину полей-указателей в структуре они освобождают при ее удалении, а вторую — нет? Если бы не вызывали free совсем — было бы понятно, типа такие стремные code guidelines; а так — ни туда, ни сюда.
багрепорт или не было
м.б. там не надо освобождать, всякое бывает
Ну посмотри на код функций UdfspVolumeTableDestroyEntry и UdfsMount, и будет тебе багрепорт, например.
При монтировании тома с файловой системой udfs для него выделяется структура (контекст), одним из полей которой является указатель на кусок памяти, используемый для кеширования разных критичных секторов. После отмонтирования контекст освобождается, все остальные структуры кроме этого куска тоже освобождаются. В чем смысл оставлять его, если заведомо нигде больше ссылок туда быть не может?
Я её не писал, в MS отрепортил? Линк?
Мне говорят что я стал жертвой пиратства, и потому не могу репортить баги (
Не знаю, не сталкивался, м.б. и баг, отрепорти.
ы?..
А триалка не даёт репортить баги?
На самом деле хуй знает, так-то я вообще под xp сижу. Да и в любом случае лень этим заниматься.
пфф, ты бы ещё в вин98 такие баги раскапывал
Я же написал — баг в загрузчике семерки.
Он несколько упорот, как ты мог заметить.
Тогда причём тут ХР? Пиши баг репорт для 7.
Не несколько