Unsoundness: `JSValue` (and others) can outlive the `Context`
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 25/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- javascript, rust
- Ambito
- tooling
Direzione di ricerca
Inizia con la riproduzione in Rust nell’issue e segui le definizioni di JSContext, JSValue, JSObject ed evaluate_script. Conferma il use-after-free dopo la fine dello scope di Context, quindi determina e documenta un approccio di ownership sicuro rispetto al lifetime. Il lavoro è completato quando la riproduzione non può accedere ai valori dopo che il suo Context è stato eliminato e il comportamento di sicurezza pertinente è coperto da un test.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Hi,
JSValue, JSObject and others are not restricted in the lifetime, therefore are able to outlive the Context, in the best case just crashing the program and in the worst case causing a use-after-free.
The following script reproduces the problem and leads to a segfault (due to an safety check in jsc itself):
use javascriptcore::*;
fn main() {
let v;
{
let ctx = JSContext::default();
v = evaluate_script(&ctx, "'Hello' + ' ' + 'World'", None, "test.js", 1)
.expect("Failed to evaluate script");
println!("Value: {:?}", v.as_string());
}
println!("Value: {:?}", v.as_string());
}
Typically, you would add PhantomData to add a phantom lifetime. However, you already hold a raw pointer to the context in these structs and the context should be a widespread object in many real-world users of your bindings, ref-counting the context in Rust with Arc might be the best option.
Kind Regards
Tim
- Lingua principale
- Rust
- Stelle
- 54
- Fork
- 4
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di endoli/javascriptcore.rs
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 25/100
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 15/100
-
Bind JSObject* functions Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 20/100
-
Windows support Aperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 20/100
endoli/javascriptcore.rs#4 · 5 commenti ·
-
Provide better Debug impls Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
Tutte le issue di endoli/javascriptcore.rs
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
-
bug core
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
-
JIT-compiled number -> Decimal conversion silently overflows instead of raising DECIMAL_OVERFLOW Apertafuzz
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
ClickHouse/ClickHouse#122114 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
linebender/vello_svg#90 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100