summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMikulas Patocka <mpatocka@redhat.com>2017-02-15 11:01:49 -0500
committerZdenek Kabelac <zkabelac@redhat.com>2017-03-27 20:50:19 +0200
commit78d004efa8a1809cea68283e6204edfa9d7c1091 (patch)
tree42dce084f7ad65b21f54b2a0a3380f457de0fb29
parent36cac41115cca875f5d120631933b5dfb3c3c524 (diff)
downloadlvm2-78d004efa8a1809cea68283e6204edfa9d7c1091.tar.gz
build: fix x32 arch
This patch fixed lvm2 compilation running on x32 arch. (Using 64bit x86 cpu features but running on 32b address space, so consuming less mem in VM). On x32 arch 'time_t' is 64bit while 'long' is 32bit.
-rw-r--r--lib/format1/import-export.c4
-rw-r--r--lib/format_text/export.c2
-rw-r--r--lib/report/report.c10
3 files changed, 8 insertions, 8 deletions
diff --git a/lib/format1/import-export.c b/lib/format1/import-export.c
index c17403127..c29527b67 100644
--- a/lib/format1/import-export.c
+++ b/lib/format1/import-export.c
@@ -128,8 +128,8 @@ int import_pv(const struct format_type *fmt, struct dm_pool *mem,
int generate_lvm1_system_id(struct cmd_context *cmd, char *s, const char *prefix)
{
- if (dm_snprintf(s, NAME_LEN, "%s%s%lu",
- prefix, cmd->hostname, time(NULL)) < 0) {
+ if (dm_snprintf(s, NAME_LEN, "%s%s" FMTu64,
+ prefix, cmd->hostname, (uint64_t)time(NULL)) < 0) {
log_error("Generated LVM1 format system_id too long");
return 0;
}
diff --git a/lib/format_text/export.c b/lib/format_text/export.c
index 899ff45cb..473275d4c 100644
--- a/lib/format_text/export.c
+++ b/lib/format_text/export.c
@@ -350,7 +350,7 @@ static int _print_header(struct cmd_context *cmd, struct formatter *f,
_utsname.version, _utsname.machine);
if (cmd->system_id && *cmd->system_id)
outf(f, "creation_host_system_id = \"%s\"", cmd->system_id);
- outf(f, "creation_time = %lu\t# %s", t, ctime(&t));
+ outf(f, "creation_time = " FMTu64 "\t# %s", (uint64_t)t, ctime(&t));
return 1;
}
diff --git a/lib/report/report.c b/lib/report/report.c
index 1421b31fa..d9880b206 100644
--- a/lib/report/report.c
+++ b/lib/report/report.c
@@ -1008,7 +1008,7 @@ static int _translate_time_items(struct dm_report *rh, struct time_info *info,
dm_pool_free(info->mem, info->ti_list);
info->ti_list = NULL;
- if (dm_snprintf(buf, sizeof(buf), "@%ld:@%ld", t1, t2) == -1) {
+ if (dm_snprintf(buf, sizeof(buf), "@" FMTd64 ":@" FMTd64, (int64_t)t1, (int64_t)t2) == -1) {
log_error("_translate_time_items: dm_snprintf failed");
return 0;
}
@@ -1063,10 +1063,10 @@ static void *_lv_time_handler_get_dynamic_value(struct dm_report *rh,
struct dm_pool *mem,
const char *data_in)
{
- time_t t1, t2;
+ int64_t t1, t2;
time_t *result;
- if (sscanf(data_in, "@%ld:@%ld", &t1, &t2) != 2) {
+ if (sscanf(data_in, "@" FMTd64 ":@" FMTd64, &t1, &t2) != 2) {
log_error("Failed to get value for parsed time specification.");
return NULL;
}
@@ -1076,8 +1076,8 @@ static void *_lv_time_handler_get_dynamic_value(struct dm_report *rh,
return NULL;
}
- result[0] = t1;
- result[1] = t2;
+ result[0] = (time_t) t1; /* Validate range for 32b arch ? */
+ result[1] = (time_t) t2;
return result;
}