Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[bug] The --uid parameter doesn't work. #694

Open
robvinson opened this issue Aug 16, 2024 · 1 comment · May be fixed by #698
Open

[bug] The --uid parameter doesn't work. #694

robvinson opened this issue Aug 16, 2024 · 1 comment · May be fixed by #698
Labels
freshissue Default label for new, untriaged issues.

Comments

@robvinson
Copy link

robvinson commented Aug 16, 2024

Describe the bug
The --uid flag introduced by #526 does not work. Maybe providing a process ID (--pid) flag could be an alternative?

Expected behavior
Attach to the application running under the UID indicated by the --uid flag.

Evidence / Logs / Screenshots
Running a command like: objection --uid 12 --name com.some.package start errors out with a stacktrace like the below. It appears that attach() doesn't actually support a UID parameter.

  File "/home/me/objection/utils/agent.py", line 250, in attach
    self.session = self.device.attach(self.pid, uid=self.config.uid)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/me/venv/lib/python3.11/site-packages/frida/core.py", line 86, in wrapper
    return f(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^
TypeError: Device.attach() got an unexpected keyword argument 'uid'

Environment (please complete the following information):

  • Frida Version: 16.4.8
  • Objection Version: master d9c989d
@robvinson robvinson added the freshissue Default label for new, untriaged issues. label Aug 16, 2024
@IPMegladon
Copy link
Collaborator

IPMegladon commented Aug 21, 2024

Thanks, I do not currently have an environment where I can test this, but do you mind checking if frida/frida#1353 (comment) works within your environment in order to narrow down where the issue might be.

@IPMegladon IPMegladon linked a pull request Sep 9, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
freshissue Default label for new, untriaged issues.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants