Re: [PATCH] grep: plug a trivial memory leak

2017-04-10 Thread Brandon Williams
On 04/09, Ævar Arnfjörð Bjarmason wrote: > Change the cleanup phase for the grep command to free the pathspec > struct that's allocated earlier in the same block, and used just a few > lines earlier. > > With "grep hi README.md" valgrind reports a loss of 239 bytes now, > down from 351. > > The r

[PATCH] grep: plug a trivial memory leak

2017-04-09 Thread Ævar Arnfjörð Bjarmason
Change the cleanup phase for the grep command to free the pathspec struct that's allocated earlier in the same block, and used just a few lines earlier. With "grep hi README.md" valgrind reports a loss of 239 bytes now, down from 351. The relevant --num-callers=40 --leak-check=full --show-leak-ki