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.

Sign up for GitHub

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

Jump to bottom

Improve the php implementation of meterpreter #714

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Improve the php implementation of meterpreter #714

wants to merge 8 commits into from

Conversation

jvoisin
Copy link
Contributor

@jvoisin jvoisin commented Sep 1, 2024

  • Add a couple of fallbacks
  • Add a bunch of proper checks for available/defined/callable functions
  • Make use of proper PRNG when available
  • Add more information when available

@jvoisin
Implement can_call_function, a better is_callable
e78bfcc 
PHP has various ways to disable functions: suhosin, disable_functions in the
configuration file, Snuffleupagus, unloading the required extension, …
meaning that sometimes functions aren't defined, aren't really a function, or
aren't callable in the current context.

This commit adds a `can_call_function` doing all those checks.
@jvoisin
Minor code simplification/golfing
e878523
@jvoisin
Add a missing @ to an unlink
297b4b8 
So that we don't get logs/errors/… should the call fail.
@jvoisin
Add a fallback to get the current user
7e7b40c
@jvoisin
Add more info to stdapi_sys_config_sysinfo when possible
3630198
@jvoisin
Add a check in case proc_open isn't callable
074c851
@jvoisin
Add a fallback to get the hostname
7f21e9a
@jvoisin
Make use of proper CSPRNG when possible
c387d2f
@dledda-r7 dledda-r7 self-assigned this Sep 16, 2024
@dledda-r7 dledda-r7 removed their assignment Sep 24, 2024
@dledda-r7 dledda-r7 self-assigned this Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@dledda-r7 dledda-r7

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jvoisin @dledda-r7