mirror of
https://github.com/isar/libmdbx.git
synced 2025-03-26 12:16:06 +08:00
mdbx: исправление регресса в пути обработки MDBX_MULTIPLE
.
Пакетная вставка значений посредством операции `MDBX_MULTIPLE` могла приводить к падениям и повреждению структуры БД. Ошибка оставалось не замеченной из-за специфических условий проявления, которые не реализовались в тестах. Проблема присутствовала во всех выпусках начиная с v0.13.1, но соответствующая ошибка не связана с конкретным коммита в истории, а является следствием нескольких доработок (шагов рефакторинга), которые суммарно привели к регрессу. Технически ошибка обусловлена не-обнулением переменной, которая не обнулялась в некотором пути выполнения и исходно не требовала обнуления, но такое обнуление потребовалось после ряда этапов оптимизации кода и рефакторинга. Основным условием проявления является пакетная вставка multi-значений в dupsort-таблицу с фиксированным размером значений, при котором набор значений соответствующий обновляемом ключу, перестаёт помещаться на вложенной странице и преобразуется/выносится во вложенное дерево страниц. Если такой вынос/преобразование происходило до исчерпания переданного набора значений, то при следующей итерации повторно производились действия соответствующие выносу данных в отдельное дерево страниц. Что могла приводить как к разыменованию неверных указателей (повреждению содержимого памяти) и/или к повреждению содержимого страниц образующих структуру БД. Исправление свелось к добавлению одной строчки кода, но также были расширены тесты для покрытия соответствующих сценариев.
This commit is contained in:
parent
db44f4ed71
commit
23a417fe19
@ -1409,6 +1409,7 @@ insert_node:;
|
||||
data[0].iov_base = ptr_disp(data[0].iov_base, data[0].iov_len);
|
||||
insert_key = insert_data = false;
|
||||
old_singledup.iov_base = nullptr;
|
||||
sub_root = nullptr;
|
||||
goto more;
|
||||
}
|
||||
}
|
||||
|
Loading…
x
Reference in New Issue
Block a user