search

rust-osdev / linked-list-allocator

Apache License 2.0
219 stars 53 forks source link

fix(deallocate_middle): advance to next in try_insert_after #63

Closed haraldh closed 2 years ago

haraldh commented 2 years ago

If the next hole node is still lower than the hole to be added, advance the cursor to the next in the list.

For this to trigger, this patch modifies the deallocate_middle() test. It requires two holes in the list before the one to be deallocated.

Signed-off-by: Harald Hoyer harald@profian.com

jamesmunns commented 2 years ago

Hey @haraldh, I think I just hit the same issue in #64, I'll be adding more tests.

Sorry for the issue!

jamesmunns commented 2 years ago

Superceded by #64, thank you for reporting @haraldh!