Skip to content

adding process#182

Merged
s1341 merged 6 commits intofrida:mainfrom
Xoffio:xo-process
Dec 17, 2024
Merged

adding process#182
s1341 merged 6 commits intofrida:mainfrom
Xoffio:xo-process

Conversation

@Xoffio
Copy link
Copy Markdown
Contributor

@Xoffio Xoffio commented Nov 26, 2024

Worked on adding the next features for Process:

  • Process.id
  • Process.arch
  • Process.platform
  • Process.pageSize
  • Process.pointerSize
  • Process.codeSigningPolicy
  • Process.mainModule
  • Process.getCurrentDir()
  • Process.getHomeDir()
  • Process.getTmpDir()
  • Process.isDebuggerAttached()
  • Process.getCurrentThreadId()
  • Process.enumerateThreads()
  • Process.findModuleByAddress(address), Process.getModuleByAddress(address), Process.findModuleByName(name), Process.getModuleByName(name)
  • Process.enumerateModules()
  • Process.findRangeByAddress(address), getRangeByAddress(address)
  • Process.enumerateRanges(protection|specifier)
  • Process.enumerateMallocRanges()
  • Process.setExceptionHandler(callback)

I will slowly add more to make the code review easier.
Tracking issue #187

@s1341
Copy link
Copy Markdown
Contributor

s1341 commented Nov 27, 2024

Thanks!

Do you want to do multiple PRs, or just multiple commits?

@Xoffio
Copy link
Copy Markdown
Contributor Author

Xoffio commented Nov 27, 2024

Thanks!

Do you want to do multiple PRs, or just multiple commits?

I would like to do multiple PRs. I can make an issue to track the main progress for Process

@Xoffio
Copy link
Copy Markdown
Contributor Author

Xoffio commented Nov 30, 2024

Have anything related to CI changed? I added some docs and now it's failing because something else I haven't even touch

@s1341
Copy link
Copy Markdown
Contributor

s1341 commented Dec 1, 2024

Could be an updated version of rust. If you don't mind, fix the errors.

Github CI is now using rustc 1.82.0
@s1341
Copy link
Copy Markdown
Contributor

s1341 commented Dec 17, 2024

Do you want to merge this?

@Xoffio
Copy link
Copy Markdown
Contributor Author

Xoffio commented Dec 17, 2024

Yes please

@s1341 s1341 merged commit ce60e74 into frida:main Dec 17, 2024
@s1341
Copy link
Copy Markdown
Contributor

s1341 commented Dec 17, 2024

Ok. Merged. Please make a tracking issue for the remaining bits.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants