Bug 1224496 (CVE-2024-35902) - VUL-0: CVE-2024-35902: kernel: net/rds: fix possible cp null dereference
Summary: VUL-0: CVE-2024-35902: kernel: net/rds: fix possible cp null dereference
Status: RESOLVED FIXED
Alias: CVE-2024-35902
Product: SUSE Security Incidents
Classification: Novell Products
Component: Incidents (show other bugs)
Version: unspecified
Hardware: Other Other
: P3 - Medium : Normal
Target Milestone: ---
Assignee: Security Team bot
QA Contact: Security Team bot
URL: https://smash.suse.de/issue/406591/
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-05-20 13:01 UTC by SMASH SMASH
Modified: 2024-06-10 12:25 UTC (History)
2 users (show)

See Also:
Found By: Security Response Team
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description SMASH SMASH 2024-05-20 13:01:49 UTC
In the Linux kernel, the following vulnerability has been resolved:

net/rds: fix possible cp null dereference

cp might be null, calling cp->cp_conn would produce null dereference

[Simon Horman adds:]

Analysis:

* cp is a parameter of __rds_rdma_map and is not reassigned.

* The following call-sites pass a NULL cp argument to __rds_rdma_map()

  - rds_get_mr()
  - rds_get_mr_for_dest

* Prior to the code above, the following assumes that cp may be NULL
  (which is indicative, but could itself be unnecessary)

	trans_private = rs->rs_transport->get_mr(
		sg, nents, rs, &mr->r_key, cp ? cp->cp_conn : NULL,
		args->vec.addr, args->vec.bytes,
		need_odp ? ODP_ZEROBASED : ODP_NOT_NEEDED);

* The code modified by this patch is guarded by IS_ERR(trans_private),
  where trans_private is assigned as per the previous point in this analysis.

  The only implementation of get_mr that I could locate is rds_ib_get_mr()
  which can return an ERR_PTR if the conn (4th) argument is NULL.

* ret is set to PTR_ERR(trans_private).
  rds_ib_get_mr can return ERR_PTR(-ENODEV) if the conn (4th) argument is NULL.
  Thus ret may be -ENODEV in which case the code in question will execute.

Conclusion:
* cp may be NULL at the point where this patch adds a check;
  this patch does seem to address a possible bug

References:
http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2024-35902
https://www.cve.org/CVERecord?id=CVE-2024-35902
https://git.kernel.org/stable/c/62fc3357e079a07a22465b9b6ef71bb6ea75ee4b
https://git.kernel.org/stable/c/6794090c742008c53b344b35b021d4a3093dc50a
https://git.kernel.org/stable/c/92309bed3c5fbe2ccd4c45056efd42edbd06162d
https://git.kernel.org/stable/c/bcd46782e2ec3825d10c1552fcb674d491cc09f9
https://git.kernel.org/stable/c/cbaac2e5488ed54833897264a5ffb2a341a9f196
https://git.kernel.org/stable/c/cfb786b03b03c5ff38882bee38525eb9987e4d14
https://git.kernel.org/stable/c/d275de8ea7be3a453629fddae41d4156762e814c
https://git.kernel.org/stable/c/d49fac38479bfdaec52b3ea274d290c47a294029
https://git.kernel.org/pub/scm/linux/security/vulns.git/plain/cve/published/2024/CVE-2024-35902.mbox
Comment 1 Joey Lee 2024-05-23 06:55:53 UTC
joeyli@linux-691t:/mnt/working/source_code-git/kernel-source> ./scripts/check-kernel-fix -s 0 CVE-2024-35902
62fc3357e079 ("net/rds: fix possible cp null dereference") merged v6.9-rc3~30^2~36
Fixes: c055fc00c07b ("net/rds: fix WARNING in rds_conn_connect_if_down") merged v6.8~19^2~5
Security fix for CVE-2024-35902 bsc#1224496 with CVSS 0
Experts candidates: mkubecek@suse.cz denis.kirjanov@suse.com davide.benini@suse.com 
..............................
NO ACTION NEEDED: All relevant branches contain the fix!

does not affect any branch. reset assignee
Comment 2 Gabriele Sonnu 2024-06-10 12:25:15 UTC
All done, closing.