-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
"index out of bounds: the len is 64 but the index is 64" when loading discord.gg #36792
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
The comment right above that line sure looks suspicious :D |
servo/ipc-channel#392 reveals that the route that causes the panic is added here: servo/components/script/script_thread.rs Lines 877 to 882 in b10fc49
New Backtrace
|
Reduced testcase: <!DOCTYPE html>
<html>
<body>
<img src="https://cdn.prod.website-files.com/6257adef93867e50d84d30e2/664754815450cb39bca27b05_Smoke.gif"> (I think a smoke gif is very fitting considering it's lighting our IPC code on fire) |
Fuzzers found the same panic message using the following canvas-based testcase: Minimal testcase: <!DOCTYPE html>
<script>
let canvas = document.createElement("canvas");
let context = canvas.getContext("2d");
let textMetrics = context.measureText("A");
let width = textMetrics.width;
let height = textMetrics.actualBoundingBoxAscent + textMetrics.actualBoundingBoxDescent;
context.drawImage(canvas, 0, 0, width, height, 0, 0, width, height);
</script> System: Linux
|
To reproduce:
./mach build && ./mach run discord.gg
Backtrace
The text was updated successfully, but these errors were encountered: