[lvm-devel] master - dmsetup: Fix multi-line concise table parsing

Alasdair Kergon agk at sourceware.org
Mon Feb 25 13:43:44 UTC 2019


Gitweb:        https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=ccfbd505fea2f259f7c89dc23b020b838363a611
Commit:        ccfbd505fea2f259f7c89dc23b020b838363a611
Parent:        eff33684f774bb732e66898a65c52163d0987b70
Author:        Alasdair G Kergon <agk at redhat.com>
AuthorDate:    Mon Feb 25 13:41:51 2019 +0000
Committer:     Alasdair G Kergon <agk at redhat.com>
CommitterDate: Mon Feb 25 13:41:51 2019 +0000

dmsetup: Fix multi-line concise table parsing

Use the correct loop variable within the loop, instead of reusing the
initial value.  Table lines after the first don't get terminated in
the right place.

Signed-off-by: Kurt Garloff <kurt at garloff.de>
---
 libdm/dm-tools/dmsetup.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/libdm/dm-tools/dmsetup.c b/libdm/dm-tools/dmsetup.c
index 2feb419..928b599 100644
--- a/libdm/dm-tools/dmsetup.c
+++ b/libdm/dm-tools/dmsetup.c
@@ -368,7 +368,7 @@ static int _parse_table_lines(struct dm_task *dmt)
 
 	do {
 		/* Identify and terminate each line */
-		if ((next_pos = strchr(_table, '\n')))
+		if ((next_pos = strchr(pos, '\n')))
 			*next_pos++ = '\0';
 		if (!_parse_line(dmt, pos, "", ++line))
 			return_0;




More information about the lvm-devel mailing list