HistoryEdit

RUSTSEC-2021-0035

quinn invalidly assumes the memory layout of std::net::SocketAddr

Issued
Package
quinn (crates.io)
Type
INFO Unsound
Keywords
#memory #layout #cast
Aliases
Details
https://github.com/quinn-rs/quinn/issues/968
CVSS Score
7.5 HIGH
CVSS Details
Attack vector
Network
Attack complexity
Low
Privileges required
None
User interaction
None
Scope
Unchanged
Confidentiality
High
Integrity
None
Availability
None
CVSS Vector
CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:H/I:N/A:N
Patched
  • ^0.5.4
  • ^0.6.2
  • >=0.7.0
Keywords
#memory #layout #cast

Description

The quinn crate has assumed std::net::SocketAddrV4 and std::net::SocketAddrV6 have the same memory layout as the system C representation sockaddr. It has simply casted the pointers to convert the socket addresses to the system representation. The standard library does not say anything about the memory layout, and this will cause invalid memory access if the standard library changes the implementation. No warnings or errors will be emitted once the change happens.