Bug 2172585 - dmstats create --filemap: error when file name contains whitespace
Summary: dmstats create --filemap: error when file name contains whitespace
Keywords:
Status: NEW
Alias: None
Product: Red Hat Enterprise Linux 9
Classification: Red Hat
Component: lvm2
Version: 9.2
Hardware: All
OS: Linux
medium
medium
Target Milestone: rc
: ---
Assignee: LVM Team
QA Contact: cluster-qe
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2023-02-22 15:42 UTC by Bryn M. Reeves
Modified: 2023-08-10 15:41 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed:
Type: Bug
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker RHELPLAN-149687 0 None None None 2023-02-22 15:43:49 UTC

Description Bryn M. Reeves 2023-02-22 15:42:58 UTC
Description of problem:
The dmstats create --filemap operation creates statistics regions that map the extents of a file in a file system and groups them together to aggregate statistics for the file.

This fails when the file name contains whitespace characters:

  # dmstats create --filemap RHEL\ 8.4\ Clone-disk1.vdi
  device-mapper: message ioctl on  (253:0) failed: Invalid argument
  Could not create regions from file /root/RHEL 8.4 Clone/RHEL 8.4 Clone-disk1.vdi.
  free(): double free detected in tcache 2
  Aborted (core dumped)

There are two problems here: the message ioctl fails since whitespace is not supported in the stats region aux_data value:

  dm message   (253:0) [ opencount flush ]  @stats_set_aux 0 DMS_GROUP=RHEL 8.4 Clone-disk1.vdi:0-19#-  [16384] (*1)
  device-mapper: message ioctl on  (253:0) failed: Invalid argument

The embedded space causes stats_set_aux value to be treated as three separate arguments: "DMS_GROUP=RHEL", "8.4", "Clone-disk1.vdi:0-19#-"

The double free is then triggered because the dmstats error path destroys the dm_bitmap used to track group members on error, and this is also freed by the later call to dm_stats_destroy for the stats handle used for the operation.

Renaming the file to use '_' instead of space avoids the error:

  # dmstats create --filemap RHEL_8.4_Clone-disk1.vdi
  RHEL_8.4_Clone-disk1.vdi: Created new group with 4 region(s) as group ID 0.

Version-Release number of selected component (if applicable):
Reproduced on RHEL8, Fedora & current git
device-mapper-1.02.175-9.fc37.x86_64

How reproducible:
100%

Steps to Reproduce:
1. Create a file with space characters in the name
2. Attempt to create stats regions with "dmstats create --filemap ..."


Actual results:
Error setting aux data and double free/abort.

Expected results:
dmstats should be able to map the regions of the file.

Additional info:
The file name is stored in aux_data as a group alias for the group of regions that map the file's extents. One way to avoid the problem would be to escape space characters as "\x20" when passing the alias to the stats_set_aux message.


Note You need to log in before you can comment on or make changes to this bug.