diff mbox

common/rc: use /bin/bash in _user_do()

Message ID 1484325496-3995-1-git-send-email-hch@lst.de (mailing list archive)
State New, archived
Headers show

Commit Message

Christoph Hellwig Jan. 13, 2017, 4:38 p.m. UTC 
Without this the epressions in generic/256 will fail on a system
where /bin/sh is the Default (e.g. modern Debian versions).

Signed-off-by: Christoph Hellwig <hch@lst.de>
---
 common/rc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Eryu Guan Jan. 15, 2017, 5:57 a.m. UTC | #1 
On Fri, Jan 13, 2017 at 05:38:16PM +0100, Christoph Hellwig wrote:
> Without this the epressions in generic/256 will fail on a system
> where /bin/sh is the Default (e.g. modern Debian versions).
> 
> Signed-off-by: Christoph Hellwig <hch@lst.de>

I dropped the generic/256 fix and applied this one.

Thanks,
Eryu
--
To unsubscribe from this list: send the line "unsubscribe fstests" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/common/rc b/common/rc
index 37d0fc8..63cabfd 100644
--- a/common/rc
+++ b/common/rc
@@ -1809,7 +1809,7 @@  _user_do()
 	then
 	echo $1 | /bin/bash "su $qa_user 2>&1" | _filter_user_do
     else
-	echo $1 | su $qa_user 2>&1 | _filter_user_do
+	echo $1 | su -s /bin/bash $qa_user 2>&1 | _filter_user_do
     fi
 }