diff options
| author | Martin Pitt <martin.pitt@ubuntu.com> | 2014-12-15 13:06:48 +0100 |
|---|---|---|
| committer | David Herrmann <dh.herrmann@gmail.com> | 2014-12-15 13:50:11 +0100 |
| commit | e17fb3c1dd21929119968325c46ab98630446032 (patch) | |
| tree | 0013f98e139068acda0f1de20d96e0bc66175b70 /src/shared/time-dst.c | |
| parent | d47f6ca5f9b7a0b400d8bdb050151a0284fb4bdb (diff) | |
shared: time-dst: Avoid buffer overflow
Commit 681f9718 introduced an additional null terminator for the zone names.
Increase the allocation of "transitions" to actually make room for this.
Diffstat (limited to 'src/shared/time-dst.c')
| -rw-r--r-- | src/shared/time-dst.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/shared/time-dst.c b/src/shared/time-dst.c index 926d22b94b..1ce6f721b7 100644 --- a/src/shared/time-dst.c +++ b/src/shared/time-dst.c @@ -183,7 +183,8 @@ read_again: return -EINVAL; } - transitions = malloc0(total_size + tzspec_len); + /* leave space for additional zone_names zero terminator */ + transitions = malloc0(total_size + tzspec_len + 1); if (transitions == NULL) return -EINVAL; |