Skip to content

Fix path shortening - #1185

Open
prettyroseslover wants to merge 1 commit into
servo:mainfrom
prettyroseslover:fix-path-shortening-after-fuzzing
Open

prettyroseslover wants to merge 1 commit into
servo:mainfrom
prettyroseslover:fix-path-shortening-after-fuzzing

Conversation

@prettyroseslover

@prettyroseslover prettyroseslover commented Nov 12, 2026 •

Copy link
Copy Markdown

During fuzz testing I found a bug in path shortening.
Minimal snippet to reproduce:

use url::Url;

fn main() {
    let base_str = "file:eeeeeeeeeeexmlns=\"http://U:";
    let base_url = Url::parse(base_str).unwrap();
    let raw_str = "../../index.html";
    Url::join(&base_url, raw_str).unwrap();
}

Fixed it and added a unit-test

This branch has not been deployed

No deployments
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.

1 participant