]> sourceware.org Git - lvm2.git/commit
libdm: use destination size as limit in dm_bit_copy()
authorBryn M. Reeves <bmr@redhat.com>
Mon, 12 Dec 2016 20:28:29 +0000 (20:28 +0000)
committerBryn M. Reeves <bmr@redhat.com>
Wed, 14 Dec 2016 11:28:11 +0000 (11:28 +0000)
commit35791689ba5ef95da45290fd12ce9cff55c86258
treeddff799060f1c9c8b1a71bc94dd377e62fc63c97
parent0f98d5c2e6d103a46bb5eca75ac496622933c475
libdm: use destination size as limit in dm_bit_copy()

The dm_bit_copy() macro uses the source (bs1) bitset size as the
limit for memcpy:

    memcpy((bs1) + 1, (bs2) + 1, ((*(bs1) / DM_BITS_PER_INT) + 1)..)

This is safe if the destination bitset is smaller than the source,
or if the two bitsets are of the same size.

With a destination that is larger (e.g. when resizing a bitmap to
add more capacity), the memcpy will overrun the source bitset and
set garbage bits in the destination.

There are nine uses of the macro currently (8 in libdm/regex, and
1 in daemons/cmirrord): in each case the two bitsets are always of
equal size so the behaviour is unchanged.

Fix the macro to use bs2's size to simplify resizing bitsets and
avoid the need for another copy macro.
libdm/libdevmapper.h
This page took 0.03751 seconds and 5 git commands to generate.