diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2012-06-21 21:19:28 +0200 |
---|---|---|
committer | Zdenek Kabelac <zkabelac@redhat.com> | 2012-06-22 13:23:03 +0200 |
commit | 6f3cd63551db013e3cf5dfcebc41115322d5cfe9 (patch) | |
tree | 0dc6d844cc137cb3e665f79ac78ead4db12bd697 /tools | |
parent | da42ee3a1f0280742e87a96f5d0694fb869a23e0 (diff) | |
download | lvm2-6f3cd63551db013e3cf5dfcebc41115322d5cfe9.tar.gz lvm2-6f3cd63551db013e3cf5dfcebc41115322d5cfe9.tar.xz lvm2-6f3cd63551db013e3cf5dfcebc41115322d5cfe9.zip |
cleanup: replace memset with struct initilization
Simplifies the code, properly detects too long socket paths,
drops unused parameter.
Diffstat (limited to 'tools')
-rw-r--r-- | tools/pvscan.c | 8 | ||||
-rw-r--r-- | tools/toollib.c | 1 |
2 files changed, 3 insertions, 6 deletions
diff --git a/tools/pvscan.c b/tools/pvscan.c index f0e74081..62034c9b 100644 --- a/tools/pvscan.c +++ b/tools/pvscan.c @@ -28,9 +28,9 @@ static void _pvscan_display_single(struct cmd_context *cmd, char uuid[64] __attribute__((aligned(8))); unsigned vg_name_len = 0; - char pv_tmp_name[NAME_LEN] = { 0, }; - char vg_tmp_name[NAME_LEN] = { 0, }; - char vg_name_this[NAME_LEN] = { 0, }; + char pv_tmp_name[NAME_LEN] = { 0 }; + char vg_tmp_name[NAME_LEN] = { 0 }; + char vg_name_this[NAME_LEN] = { 0 }; /* short listing? */ if (arg_count(cmd, short_ARG) > 0) { @@ -49,8 +49,6 @@ static void _pvscan_display_single(struct cmd_context *cmd, /* return; */ } - memset(pv_tmp_name, 0, sizeof(pv_tmp_name)); - vg_name_len = strlen(pv_vg_name(pv)) + 1; if (arg_count(cmd, uuid_ARG)) { diff --git a/tools/toollib.c b/tools/toollib.c index d5ad805a..361052b3 100644 --- a/tools/toollib.c +++ b/tools/toollib.c @@ -674,7 +674,6 @@ static int _process_all_devs(struct cmd_context *cmd, void *handle, dm_list_init(&pv_dummy.tags); dm_list_init(&pv_dummy.segments); pv_dummy.dev = dev; - pv_dummy.fmt = NULL; pv = &pv_dummy; } ret = process_single_pv(cmd, NULL, pv, handle); |