diff options
author | Alex Vandiver <alexmv@dropbox.com> | 2017-10-04 06:27:46 (GMT) |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2017-10-04 09:58:53 (GMT) |
commit | 2a387b17c5bc5e0872bed352a41a2b312ea86f9b (patch) | |
tree | 28e39bf1dc1bbff4003023509b0222b86b4c6f2c /entry.c | |
parent | dcdb71f1599734ae46870d3eca11e2093bbd7520 (diff) | |
download | git-2a387b17c5bc5e0872bed352a41a2b312ea86f9b.zip git-2a387b17c5bc5e0872bed352a41a2b312ea86f9b.tar.gz git-2a387b17c5bc5e0872bed352a41a2b312ea86f9b.tar.bz2 |
fsmonitor: read entirety of watchman output
In Perl, setting $/ sets the string that is used as the "record
separator," which sets the boundary that the `<>` construct reads to.
Setting `local $/ = 0666;` evaluates the octal, getting 438, and
stringifies it. Thus, the later read from `<CHLD_OUT>` stops as soon
as it encounters the string "438" in the watchman output, yielding
invalid JSON; repositories containing filenames with SHA1 hashes are
able to trip this easily.
Set `$/` to undefined, thus slurping all output from watchman. Also
close STDIN which is provided to watchman, to better guarantee that we
cannot deadlock with watchman while both attempting to read.
Signed-off-by: Alex Vandiver <alexmv@dropbox.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'entry.c')
0 files changed, 0 insertions, 0 deletions