btreplay: fix device IO remap functionality
authorIgnat Korchagin <ignat@cloudflare.com>
Mon, 16 Sep 2019 16:30:23 +0000 (10:30 -0600)
committerJens Axboe <axboe@kernel.dk>
Mon, 16 Sep 2019 16:30:23 +0000 (10:30 -0600)
commit667ac92a1a72b6038f0fae4acfc6a280fd69697b
tree2a5e5280e1ad47ee17fc958979ae2e2d74675de9
parenta7263b8fb22f07f4f1a3ec54f0c37193c5908b22
btreplay: fix device IO remap functionality

Commit dd093eb1c48e ("Fix warnings on newer gcc") moved string buffers holding
device names during map file parse stage to stack. However, only pointers to
them are being stored in the allocated "struct map_dev" structure. These
pointers are invalid outside of scope of this function and in a different
thread context. Also "release_map_devs" function still tries to "free" them
later as if they were allocated on the heap.

Moving the buffers back to the heap by instructing "fscanf" to allocate them
while parsing the file.

Alternatively, we could redefine the "struct map_dev" to include the whole
buffers instead of just pointers to them and free them as part of releasing the
whole "struct map_dev".

Fixes: dd093eb1c48e ("Fix warnings on newer gcc")
Signed-off-by: Ignat Korchagin <ignat@cloudflare.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
btreplay/btreplay.c