Bash leaks heredoc fd to child processes

2017-01-17 Thread Siteshwar Vashisht
Hello, Bash leaks heredoc fd to child processes if heredoc string contains command substitution. Reproducer code : $ cat test.sh #!/bin/bash cat

Re: Bash leaks heredoc fd to child processes

2017-01-17 Thread Chet Ramey
On 1/17/17 8:24 AM, Siteshwar Vashisht wrote: > Hello, > > Bash leaks heredoc fd to child processes if heredoc string contains command > substitution. > > Reproducer code : What is `pvs'? -- ``The lyf so short, the craft so long to lerne.'' - Chaucer ``Ars longa, vita brevis'

Re: Bash leaks heredoc fd to child processes

2017-01-17 Thread John McKown
On Tue, Jan 17, 2017 at 10:06 AM, Chet Ramey wrote: > On 1/17/17 8:24 AM, Siteshwar Vashisht wrote: > > Hello, > > > > Bash leaks heredoc fd to child processes if heredoc string contains > command substitution. > > > > Reproducer code : > > What is `pvs'? > > ​Probably a part of LVM (Logical Volu

Re: Bash leaks heredoc fd to child processes

2017-01-17 Thread Siteshwar Vashisht
- Original Message - > From: "John McKown" > To: "Chester Ramey" > Cc: bug-bash@gnu.org, "Siteshwar Vashisht" > Sent: Tuesday, January 17, 2017 5:14:44 PM > Subject: Re: Bash leaks heredoc fd to child processes > > ​Probably a part of LVM (Logical Volume Manager), > > VS(8) > Sys

Re: Bash leaks heredoc fd to child processes

2017-01-17 Thread Chet Ramey
On 1/17/17 12:38 PM, Siteshwar Vashisht wrote: >> NAME >>pvs — report information about physical volumes > > Yes, it's the utility used in my reproducer. I used it because it was verbose > about the descriptor leak. It's ok, I was able to reproduce it using something else. -- ``The l

Re: Bash leaks heredoc fd to child processes

2017-01-17 Thread Chet Ramey
On 1/17/17 8:24 AM, Siteshwar Vashisht wrote: > Hello, > > Bash leaks heredoc fd to child processes if heredoc string contains command > substitution. Thanks for the report. Here's a simpler patch. Chet -- ``The lyf so short, the craft so long to lerne.'' - Chaucer ``Ars longa

Re: Bash leaks heredoc fd to child processes

2017-01-17 Thread Siteshwar Vashisht
- Original Message - > From: "Chet Ramey" > To: "Siteshwar Vashisht" , bug-bash@gnu.org > Cc: "chet ramey" > Sent: Tuesday, January 17, 2017 7:42:07 PM > Subject: Re: Bash leaks heredoc fd to child processes > > Thanks for the report. Here's a simpler patch. Looks good to me. Thanks!