Skip to content

Feature/add uid to ps everywhere#210

Open
itayfoT wants to merge 2 commits intoEC-DIGIT-CSIRC:mainfrom
envoidshield:feature/add-uid-to-ps-everywhere
Open

Feature/add uid to ps everywhere#210
itayfoT wants to merge 2 commits intoEC-DIGIT-CSIRC:mainfrom
envoidshield:feature/add-uid-to-ps-everywhere

Conversation

@itayfoT
Copy link

@itayfoT itayfoT commented Nov 5, 2025

Add UID Extraction to ps_everywhere Analyzer

Summary

Adds User ID (UID) field to ps_everywhere analyzer output for security analysis and forensic investigations.

Changes

  • Extract UID from ps.txt, psthread.txt, spindump-nosymbols.txt, and logarchive (euid)
  • Add _sanitize_uid() helper to filter invalid placeholder values (0xAAAAAAAA, 0xFFFFFFFF)
  • Update deduplication logic to consider UID - same process with different UIDs tracked separately
  • Set uid: None for sources without UID information

Output

{
  "data": {
    "source": "ps.txt",
    "uid": 0
  }
}

@dario-br
Copy link
Contributor

Hey @itayfoT, thanks for the PR. Very much appreciated. But would you mind to remove the changes related to orjson in it? You have already opened another one PR #209 on that topic.

- Add UID field to all ps_everywhere output entries
- Extract UID from ps.txt, psthread.txt, spindump-nosymbols.txt, and logarchive (euid)
- Add _sanitize_uid() helper to filter invalid placeholder UIDs (0xAAAAAAAA, 0xFFFFFFFF)
- Update deduplication logic to consider UID as part of uniqueness
- Same process with different UIDs now tracked as separate entries
- Use None for missing/invalid UIDs (not 0)
- All tests pass successfully
- Add UID extraction with sanitization for invalid placeholder values
- Add PID and PPID to all process entries
- Add parent process name (PPNAME) resolution
- Build PID to process name mapping from multiple sources
- Update deduplication to consider UID, PID, and PPID
- Fix code style issues
@itayfoT itayfoT force-pushed the feature/add-uid-to-ps-everywhere branch from 718eabb to 4ad5b06 Compare November 14, 2025 10:55
@itayfoT
Copy link
Author

itayfoT commented Nov 14, 2025

Hi @dario-br thank you for your response, removed

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.

3 participants