diff options
| author | Junio C Hamano <gitster@pobox.com> | 2021-12-10 14:35:15 -0800 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2021-12-10 14:35:15 -0800 |
| commit | bb47eee9df438ea1934effb1fb8877deb4852821 (patch) | |
| tree | bc7340a9e776b7a1c06943508d7c1d7b450a06ff /pager.c | |
| parent | 670703e9d667f28221bfaa8baf75e90f0ba5ddca (diff) | |
| parent | f917f57f40a22b64ab5354e99dbf32bb48d9fc0a (diff) | |
| download | git-bb47eee9df438ea1934effb1fb8877deb4852821.tar.xz | |
Merge branch 'em/missing-pager'
When a non-existent program is given as the pager, we tried to
reuse an uninitialized child_process structure and crashed, which
has been fixed.
* em/missing-pager:
pager: fix crash when pager program doesn't exist
Diffstat (limited to 'pager.c')
| -rw-r--r-- | pager.c | 4 |
1 files changed, 3 insertions, 1 deletions
@@ -8,7 +8,7 @@ #define DEFAULT_PAGER "less" #endif -static struct child_process pager_process = CHILD_PROCESS_INIT; +static struct child_process pager_process; static const char *pager_program; /* Is the value coming back from term_columns() just a guess? */ @@ -124,6 +124,8 @@ void setup_pager(void) setenv("GIT_PAGER_IN_USE", "true", 1); + child_process_init(&pager_process); + /* spawn the pager */ prepare_pager_args(&pager_process, pager); pager_process.in = -1; |
