Hi,

the currently committed patch looks insufficient, because

    echo $CONFIDENTIAL | process -

is fine (echo is a shell builtin)
but

    su $user -c "echo $CONFIDENTIAL | process -"

is not, because $CONFIDENTIAL is passed as a command line parameter to su.


Attached is a completely untested patch versus current svn, that could
fix the problem and also simplifies things a lot. That patch itself
looks very ugly because its a patch for a patch :-)

It's the way I would have written it, but no warranties that it still
does what it should do

Andreas

Attachment: 625868-after-r634.diff
Description: application/pgp-keys

Reply via email to