dspinellis / dgsh

Shell supporting pipelines to and from multiple processes
http://www.spinellis.gr/sw/dgsh/
Other
323 stars 23 forks source link

Core dump when reading from subshell #59

Closed dspinellis closed 7 years ago

dspinellis commented 7 years ago
[dds@stereo coreutils]$ dgsh -c '(echo a) | cat'
dgsh: line 1: 17970 Done                    ( echo a )
     17971 Segmentation fault      (core dumped) | cat

Group commands also fail in the same way.

[dds@stereo coreutils]$ dgsh -c '{ echo a ;} | cat'
dgsh: line 1: 18055 Done                    { echo a; }
     18056 Segmentation fault      (core dumped) | cat
mfragkoulis commented 7 years ago

From commit https://github.com/mfragkoulis/bash/commit/7fecf3398f93220d2bceb80282c3bf1167b233b0 and beyond (new branch: compatibility) addresses this issue not only for subshell and group command, but for all non-dgsh compatible types of commands in the style of dgsh-wrap bash -c 'command'. Quoting is an important concern. More work might be required in this direction.