Description
In the Linux kernel, the following vulnerability has been resolved: xfrm: state: fix out-of-bounds read during lookup lookup and resize can run in parallel. The xfrm_state_hash_generation seqlock ensures a retry, but the hash functions can observe a hmask value that is too large for the new hlist array. rehash does: rcu_assign_pointer(net->xfrm.state_bydst, ndst) [..] net->xfrm.state_hmask = nhashmask; While state lookup does: h = xfrm_dst_hash(net, daddr, saddr, tmpl->reqid, encap_family); hlist_for_each_entry_rcu(x, net->xfrm.state_bydst + h, bydst) { This is only safe in case the update to state_bydst is larger than net->xfrm.xfrm_state_hmask (or if the lookup function gets serialized via state spinlock again). Fix this by prefetching state_hmask and the associated pointers. The xfrm_state_hash_generation seqlock retry will ensure that the pointer and the hmask will be consistent. The existing helpers, like xfrm_dst_hash(), are now unsafe for RCU side, add lockdep assertions to document that they are only safe for insert side. xfrm_state_lookup_byaddr() uses the spinlock rather than RCU. AFAICS this is an oversight from back when state lookup was converted to RCU, this lock should be replaced with RCU in a future patch.
Affected products
Version statuses are supplied by the assigning organization. Consult the original record for detailed ranges and platform conditions.
Linux Linux
Default version status: unaffected
c2f672fc94642bae96821a393f342edcfa9794a6 to < 80610185e9dd2ef7c0a4818a47390a8655b3dc27affectedc2f672fc94642bae96821a393f342edcfa9794a6 to < b86dc510308d7a8955f3f47a4fea4bef887653e4affectedc2f672fc94642bae96821a393f342edcfa9794a6 to < a16871c7832ea6435abb6e0b58289ae7dcb7e4fcaffectedc2f672fc94642bae96821a393f342edcfa9794a6 to < dd4c2a174994238d55ab54da2545543d36f4e0d0affectedc2f672fc94642bae96821a393f342edcfa9794a6 to < e952837f3ddb0ff726d5b582aa1aad9aa38d024daffected
Linux Linux
Default version status: affected
4.9affected0 to < 4.9unaffected6.1.191 to ≤ 6.1.*unaffected6.6.120 to ≤ 6.6.*unaffected6.12.13 to ≤ 6.12.*unaffected6.13.2 to ≤ 6.13.*unaffected6.14 to ≤ *unaffected
References
- https://git.kernel.org/stable/c/80610185e9dd2ef7c0a4818a47390a8655b3dc27 ↗
- https://git.kernel.org/stable/c/b86dc510308d7a8955f3f47a4fea4bef887653e4 ↗
- https://git.kernel.org/stable/c/a16871c7832ea6435abb6e0b58289ae7dcb7e4fc ↗
- https://git.kernel.org/stable/c/dd4c2a174994238d55ab54da2545543d36f4e0d0 ↗
- https://git.kernel.org/stable/c/e952837f3ddb0ff726d5b582aa1aad9aa38d024d ↗