[lvm-devel] master - cleanup: dmeventd reorder _fill_device_data
Zdenek Kabelac
zkabelac at fedoraproject.org
Fri Apr 18 14:55:31 UTC 2014
Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=4fb588c34ee9b64333ad39ab039d657cf6e958ab
Commit: 4fb588c34ee9b64333ad39ab039d657cf6e958ab
Parent: 6b701c3a48aede064850f7b3d16021d60fa8e541
Author: Zdenek Kabelac <zkabelac at redhat.com>
AuthorDate: Fri Apr 18 11:37:17 2014 +0200
Committer: Zdenek Kabelac <zkabelac at redhat.com>
CommitterDate: Fri Apr 18 16:38:51 2014 +0200
cleanup: dmeventd reorder _fill_device_data
Just simplify the function.
---
daemons/dmeventd/dmeventd.c | 12 ++++--------
1 files changed, 4 insertions(+), 8 deletions(-)
diff --git a/daemons/dmeventd/dmeventd.c b/daemons/dmeventd/dmeventd.c
index 4f059b9..a907ca6 100644
--- a/daemons/dmeventd/dmeventd.c
+++ b/daemons/dmeventd/dmeventd.c
@@ -404,6 +404,7 @@ static int _fill_device_data(struct thread_status *ts)
{
struct dm_task *dmt;
struct dm_info dmi;
+ int ret = 0;
if (!ts->device.uuid)
return 0;
@@ -426,16 +427,11 @@ static int _fill_device_data(struct thread_status *ts)
ts->device.major = dmi.major;
ts->device.minor = dmi.minor;
-
- dm_task_destroy(dmt);
- return 1;
-
- fail:
+ ret = 1;
+fail:
dm_task_destroy(dmt);
- dm_free(ts->device.name);
- ts->device.name = NULL;
- return 0;
+ return ret;
}
/*
More information about the lvm-devel
mailing list