diff options
| author | Brian Behlendorf <[email protected]> | 2016-02-05 18:41:22 -0500 |
|---|---|---|
| committer | Brian Behlendorf <[email protected]> | 2016-02-08 09:37:55 -0800 |
| commit | eea9309423d577d62cf62d1f98444e22e264ef15 (patch) | |
| tree | 823e293a295b6ba39490f874decacc628a9056d3 /lib/libzfs/libzfs_util.c | |
| parent | 6b42ea85903b5d96eddbe364e71636e6802d8da0 (diff) | |
Prevent zpool_find_vdev() from truncating vdev path
When extracting tokens from the string strtok(2) is allowed to modify
the passed buffer. Therefore the zfs_strcmp_pathname() function must
make a copy of the passed string before passing it to strtok(3).
Signed-off-by: Brian Behlendorf <[email protected]>
Signed-off-by: Don Brady <[email protected]>
Closes #4312
Diffstat (limited to 'lib/libzfs/libzfs_util.c')
| -rw-r--r-- | lib/libzfs/libzfs_util.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/lib/libzfs/libzfs_util.c b/lib/libzfs/libzfs_util.c index fa55fd3a6..65b04c59a 100644 --- a/lib/libzfs/libzfs_util.c +++ b/lib/libzfs/libzfs_util.c @@ -1029,16 +1029,18 @@ zfs_strcmp_pathname(char *name, char *cmp, int wholedisk) int path_len, cmp_len; char path_name[MAXPATHLEN]; char cmp_name[MAXPATHLEN]; - char *dir; + char *dir, *dup; /* Strip redundant slashes if one exists due to ZPOOL_IMPORT_PATH */ memset(cmp_name, 0, MAXPATHLEN); - dir = strtok(cmp, "/"); + dup = strdup(cmp); + dir = strtok(dup, "/"); while (dir) { strcat(cmp_name, "/"); strcat(cmp_name, dir); dir = strtok(NULL, "/"); } + free(dup); if (name[0] != '/') return (zfs_strcmp_shortname(name, cmp_name, wholedisk)); |