Skip to content

Commit

Permalink
Merge pull request #3389 from lplewa/signed
Browse files Browse the repository at this point in the history
test: do not store -1 value in unsigned variable
  • Loading branch information
marcinslusarz authored Oct 26, 2018
2 parents 41e08aa + 228762a commit 9476bdf
Showing 1 changed file with 8 additions and 3 deletions.
11 changes: 8 additions & 3 deletions src/test/pmem_deep_persist/pmem_deep_persist.c
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,12 @@ main(int argc, char *argv[])
int ret = -1;

path = argv[1];
persist_size = ATOULL(argv[3]);
ssize_t tmp = ATOLL(argv[3]);
if (tmp < 0)
persist_size = UINT64_MAX;
else
persist_size = (size_t)tmp;

offset = ATOULL(argv[4]);

switch (*argv[2]) {
Expand All @@ -79,7 +84,7 @@ main(int argc, char *argv[])
UT_FATAL("!pmem_map_file");
}

if (persist_size == -1)
if (persist_size == UINT64_MAX)
persist_size = mapped_len;
ret = pmem_deep_persist(addr + offset, persist_size);

Expand All @@ -97,7 +102,7 @@ main(int argc, char *argv[])
UT_ASSERTne(addr, MAP_FAILED);
CLOSE(fd);

if (persist_size == -1)
if (persist_size == UINT64_MAX)
persist_size = file_size;
ret = pmem_deep_persist(addr + offset, persist_size);

Expand Down

0 comments on commit 9476bdf

Please sign in to comment.