Skip to content

Conversation

@mzfr
Copy link

@mzfr mzfr commented Jan 14, 2026

Copy link
Contributor

@giacomo-doyensec giacomo-doyensec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @mzfr, thanks for your contribution!
The overall testbed looks fine I just added some suggestions in the comments down below.

- Run Vulnerable Version Only

```bash
docker-compose up llama-factory-vulnerable
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
docker-compose up llama-factory-vulnerable
docker compose up llama-factory-vulnerable

- Run Safe/Patched Version Only

```bash
docker-compose up llama-factory-safe
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
docker-compose up llama-factory-safe
docker compose up llama-factory-safe

- Run Both Versions Simultaneously

```bash
docker-compose up
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
docker-compose up
docker compose up

Reference PoC: https://gist.github.com/superboy-zjc/f2d2b93ae511c445ba97e144b70e534d

```sh
uv run poc_verify.py --url http://localhost:7860 --cmd "echo poc-vuln"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
uv run poc_verify.py --url http://localhost:7860 --cmd "echo poc-vuln"
uv run --with requests poc_verify.py --url http://localhost:<port> --cmd "echo poc-vuln"

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