diff options
| author | Zdenek Kabelac <zkabelac@redhat.com> | 2011-04-09 19:05:23 +0000 |
|---|---|---|
| committer | Zdenek Kabelac <zkabelac@redhat.com> | 2011-04-09 19:05:23 +0000 |
| commit | c67d2b4dd4941a4119e4c4ad1fdd8fb9d161cbf7 (patch) | |
| tree | 5c1798d594c0bcd2e3830485bde0b3be12028ff1 /lib/metadata/mirror.c | |
| parent | a1eba521e3cd652a8e72e662866e593d6f949fc2 (diff) | |
| download | lvm2-c67d2b4dd4941a4119e4c4ad1fdd8fb9d161cbf7.tar.gz lvm2-c67d2b4dd4941a4119e4c4ad1fdd8fb9d161cbf7.tar.xz lvm2-c67d2b4dd4941a4119e4c4ad1fdd8fb9d161cbf7.zip | |
Fix incorrect tests for dm_snprintf() failure
As the memory is preallocated based on arg size in these cases,
the error would be quite hard to trigger here anyway.
Diffstat (limited to 'lib/metadata/mirror.c')
| -rw-r--r-- | lib/metadata/mirror.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/metadata/mirror.c b/lib/metadata/mirror.c index e4aa808c..32b236cf 100644 --- a/lib/metadata/mirror.c +++ b/lib/metadata/mirror.c @@ -1787,7 +1787,7 @@ static struct logical_volume *_set_up_mirror_log(struct cmd_context *cmd, } else if ((lv_name = strstr(lv->name, MIRROR_SYNC_LAYER))) { len = lv_name - lv->name + 1; if (!(tmp_name = alloca(len)) || - !dm_snprintf(tmp_name, len, "%s", lv->name)) { + (dm_snprintf(tmp_name, len, "%s", lv->name) < 0)) { log_error("mirror log name allocation failed"); return 0; } |