diff options
| author | David Sterba <dsterba@suse.cz> | 2013-07-09 18:39:24 +0200 |
|---|---|---|
| committer | David Sterba <dsterba@suse.cz> | 2013-08-09 14:32:35 +0200 |
| commit | a8c9a2087a81c291e6a8a844fd2aaea561f64f1f (patch) | |
| tree | bbaf8ba32c0d27a216faeab33f4e94e8f43a6e1d /btrfs-list.c | |
| parent | 82ddde496a3c10e4f48363c578c9c2510d3a48e7 (diff) | |
btrfs-progs: use reentrant localtime
localtime may return NULL (when an error is detected eg. after setting
tzname), followed by a segfault when the values is about to be used.
localtime_r works, does not set tzname and does not return NULL.
Signed-off-by: David Sterba <dsterba@suse.cz>
Signed-off-by: Chris Mason <chris.mason@fusionio.com>
Diffstat (limited to 'btrfs-list.c')
| -rw-r--r-- | btrfs-list.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/btrfs-list.c b/btrfs-list.c index 9deddd57..a6902c37 100644 --- a/btrfs-list.c +++ b/btrfs-list.c @@ -1337,10 +1337,12 @@ static void print_subvolume_column(struct root_info *subv, printf("%llu", subv->top_id); break; case BTRFS_LIST_OTIME: - if (subv->otime) - strftime(tstr, 256, "%Y-%m-%d %X", - localtime(&subv->otime)); - else + if (subv->otime) { + struct tm tm; + + localtime_r(&subv->otime, &tm); + strftime(tstr, 256, "%Y-%m-%d %X", &tm); + } else strcpy(tstr, "-"); printf("%s", tstr); break; |