Skip to content

fix(model): Propagate untrusted host through HttpHost#<init>#259

Open
misonijnik wants to merge 1 commit into
mainfrom
misonijnik/model-httphost
Open

fix(model): Propagate untrusted host through HttpHost#<init>#259
misonijnik wants to merge 1 commit into
mainfrom
misonijnik/model-httphost

Conversation

@misonijnik

Copy link
Copy Markdown
Member

No description provided.

…Name field

Apache HttpComponents core5 HttpHost wraps a hostname string; SSRF sinks
such
as HttpAsyncRequester.connect(HttpHost, ...) consume a HttpHost built
from an
untrusted host. Model this field-sensitively: copy the constructor's
hostname
argument into the object's `hostName` virtual field (base `this` + field
modifier), not the whole object. Exact per-overload signatures pick the
hostname
arg (arg0 for (String) and (String,int); arg1 for (String scheme, String
host,
int port)) — no arg(*), no <rule-storage>, no whole-object `to: this`.
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