]> sourceware.org Git - lvm2.git/commit
libdm: do not add params for resume and remove
authorZdenek Kabelac <zkabelac@redhat.com>
Fri, 16 Nov 2018 14:54:09 +0000 (15:54 +0100)
committerZdenek Kabelac <zkabelac@redhat.com>
Fri, 16 Nov 2018 23:30:50 +0000 (00:30 +0100)
commit10e191fd124e89d40518a7cba5d61c47be5a0969
tree2aa51808ab36ed582b6c18ae9cdb05e41cb6a123
parentd44bfe90f1b609d23566ba26c4d51ba167f2368d
libdm: do not add params for resume and remove

DM_DEVICE_CREATE with table is doing several ioctl operations,
however only some of then takes parameters.
Since _create_and_load_v4() reused already existing dm task from
DM_DEVICE_RELOAD it has also kept passing its table parameters
to DM_DEVICE_RESUME ioctl - but this ioctl is supposed to not take
any argument and thus there is no wiping of passed data - and
since kernel returns buffer and shortens dmi->data_size accordingly,
anything past returned data size remained uncleared in zfree()
function.

This has problem if the user used dm_task_secure_data (i.e. cryptsetup),
as in this case binary expact secured data are erased from main memory
after use, but they may have been left in place.

This patch is also closing the possible hole for error path,
which also reuse same dm task structure for DM_DEVICE_REMOVE.
WHATS_NEW_DM
device_mapper/ioctl/libdm-iface.c
libdm/ioctl/libdm-iface.c
This page took 0.035417 seconds and 5 git commands to generate.