Fix the block maps for data ranges

mpalmer / lvmsync

Synchronise LVM LVs across a network by sending only snapshotted changes

http://theshed.hezmatt.org/lvmsync

GNU General Public License v3.0

383 stars 61 forks source link

Fix the block maps for data ranges #18

Closed AnchorCat closed 10 years ago

AnchorCat commented 10 years ago

The existing code uses the underlying data blocks as keys, and the origin blocks as values, which is backwards. This is done correctly above for the single mapping case, so this bug only affects range mappings.

AnchorCat commented 10 years ago

This change has been obsoleted by https://github.com/mpalmer/lvmsync/pull/19.

mpalmer commented 10 years ago

I've actually merged this, because it's a problem that definitely needs fixing, and I'm not keen on the current implementation of #19, so we'll work on that one more.