Skip to content
This repository has been archived by the owner on Sep 2, 2020. It is now read-only.

Some su implementations require -c before command #32

Open
BurntBrunch opened this issue Oct 6, 2015 · 3 comments
Open

Some su implementations require -c before command #32

BurntBrunch opened this issue Oct 6, 2015 · 3 comments

Comments

@BurntBrunch
Copy link
Contributor

Cyanogen running on m7:

: fb-adb shell -r
fb-adb shell: trouble re-execing adb stub as root: Unknown id: stub

shell@m7:/ $ su whoami
Unknown id: whoami
1|shell@m7:/ $ su -c whoami
root

@jbeich
Copy link
Contributor

jbeich commented Mar 17, 2016

Can you reproduce after b230619?

@genevera
Copy link

I get this off of master:

$  fb-adb -r shell echo 'hello'
fb-adb shell: trouble re-execing adb stub as root: fb-adb stub: execvpe("su"): Invalid argument
$  fb-adb -r shell
fb-adb shell: trouble re-execing adb stub as root: fb-adb stub: execvpe("su"): Invalid argument

@genevera
Copy link

genevera commented Sep 27, 2017

I restored the function from https://github.com/facebook/fb-adb/blob/1.4.4/cmd_stub.c and it seems to be working for both shell and ctar

ping @BurntBrunch @jbeich

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants