diff options
| author |  Michał Kępień <michal@isc.org> | 2020-10-20 08:48:08 +0200 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2020-10-20 12:53:26 -0700 |
| commit | ec7967cfafd5844e33afbc1e6fb8ba1b9e5b111b (patch) | |
| tree | fbe510604f80b470f7ab46871eb1e43e7a1903d9 /builtin | |
| parent | Git 2.28 (diff) | |
| download | tgif-ec7967cfafd5844e33afbc1e6fb8ba1b9e5b111b.tar.xz | |
merge-base, xdiff: zero out xpparam_t structures
xpparam_t structures are usually zero-initialized before their specific
fields are assigned to, but there are three locations in the tree where
that does not happen. Add the missing memset() calls in order to make
initialization of xpparam_t structures consistent tree-wide and to
prevent stack garbage from being used as field values.
Signed-off-by: Michał Kępień <michal@isc.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin')
| -rw-r--r-- | builtin/merge-tree.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/builtin/merge-tree.c b/builtin/merge-tree.c index e72714a5a8..de8520778d 100644 --- a/builtin/merge-tree.c +++ b/builtin/merge-tree.c @@ -109,6 +109,7 @@ static void show_diff(struct merge_list *entry) xdemitconf_t xecfg; xdemitcb_t ecb; + memset(&xpp, 0, sizeof(xpp)); xpp.flags = 0; memset(&xecfg, 0, sizeof(xecfg)); xecfg.ctxlen = 3; |