-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Closed
Description
There are multiple places where parsing server config from URL or just trying to parse a password panics and crashes the whole application using the library.
shadowsocks-rust/crates/shadowsocks/src/config.rs
Lines 436 to 442 in c52b77d
| panic!( | |
| "{} is expecting a {} bytes key, but password: {} ({} bytes after decode)", | |
| method, | |
| enc_key.len(), | |
| password, | |
| v.len() | |
| ); |
| panic!("{method} password {password} is not base64 encoded, error: {err}"); |
(triggered by
ServerConfig::from_url("ss://2022-blake3-aes-128-gcm:[email protected]:9999"))
| let upsk = split_iter.next().expect("uPSK"); |
| panic!("iPSK {ipsk} is not base64 encoded, error: {err}"); |
One of the cases was fixed in #1762
Metadata
Metadata
Assignees
Labels
No labels