CVE-2022-49721
- Source
- https://nvd.nist.gov/vuln/detail/CVE-2022-49721
- Import Source
- https://storage.googleapis.com/osv-test-cve-osv-conversion/osv-output/CVE-2022-49721.json
- JSON Data
- https://api.test.osv.dev/v1/vulns/CVE-2022-49721
- Downstream
- Related
- Published
- 2025-02-26T02:24:34Z
- Modified
- 2025-10-16T00:44:03.201720Z
- Summary
- arm64: ftrace: consistently handle PLTs.
- Details
-
In the Linux kernel, the following vulnerability has been resolved:
arm64: ftrace: consistently handle PLTs.
Sometimes it is necessary to use a PLT entry to call an ftrace trampoline. This is handled by ftracemakecall() and ftracemakenop(), with each having almost identical logic, but this is not handled by ftracemodifycall() since its introduction in commit:
3b23e4991fb66f6d ("arm64: implement ftrace with regs")
Due to this, if we ever were to call ftracemodifycall() for a callsite which requires a PLT entry for a trampoline, then either:
a) If the old addr requires a trampoline, ftracemodifycall() will use an out-of-range address to generate the 'old' branch instruction. This will result in warnings from aarch64insngenbranchimm() and ftracemodifycode(), and no instructions will be modified. As ftracemodifycall() will return an error, this will result in subsequent internal ftrace errors.
b) If the old addr does not require a trampoline, but the new addr does, ftracemodifycall() will use an out-of-range address to generate the 'new' branch instruction. This will result in warnings from aarch64insngenbranchimm(), and ftracemodifycode() will replace the 'old' branch with a BRK. This will result in a kernel panic when this BRK is later executed.
Practically speaking, case (a) is vastly more likely than case (b), and typically this will result in internal ftrace errors that don't necessarily affect the rest of the system. This can be demonstrated with an out-of-tree test module which triggers ftracemodifycall(), e.g.
| # insmod testftrace.ko | testftrace: Function testfunction raw=0xffffb3749399201c, callsite=0xffffb37493992024 | branchimmcommon: offset out of range | branchimmcommon: offset out of range | ------------[ ftrace bug ]------------ | ftrace failed to modify | [<ffffb37493992024>] testfunction+0x8/0x38 [testftrace] | actual: 1d:00:00:94 | Updating ftrace call site to call a different ftrace function | ftrace record flags: e0000002 | (2) R | expected tramp: ffffb374ae42ed54 | ------------[ cut here ]------------ | WARNING: CPU: 0 PID: 165 at kernel/trace/ftrace.c:2085 ftracebug+0x280/0x2b0 | Modules linked in: testftrace(+) | CPU: 0 PID: 165 Comm: insmod Not tainted 5.19.0-rc2-00002-g4d9ead8b45ce #13 | Hardware name: linux,dummy-virt (DT) | pstate: 60400005 (nZCv daif +PAN -UAO -TCO -DIT -SSBS BTYPE=--) | pc : ftracebug+0x280/0x2b0 | lr : ftracebug+0x280/0x2b0 | sp : ffff80000839ba00 | x29: ffff80000839ba00 x28: 0000000000000000 x27: ffff80000839bcf0 | x26: ffffb37493994180 x25: ffffb374b0991c28 x24: ffffb374b0d70000 | x23: 00000000ffffffea x22: ffffb374afcc33b0 x21: ffffb374b08f9cc8 | x20: ffff572b8462c000 x19: ffffb374b08f9000 x18: ffffffffffffffff | x17: 6c6c6163202c6331 x16: ffffb374ae5ad110 x15: ffffb374b0d51ee4 | x14: 0000000000000000 x13: 3435646532346561 x12: 3437336266666666 | x11: 203a706d61727420 x10: 6465746365707865 x9 : ffffb374ae5149e8 | x8 : 336266666666203a x7 : 706d617274206465 x6 : 00000000fffff167 | x5 : ffff572bffbc4a08 x4 : 00000000fffff167 x3 : 0000000000000000 | x2 : 0000000000000000 x1 : ffff572b84461e00 x0 : 0000000000000022 | Call trace: | ftracebug+0x280/0x2b0 | ftracereplacecode+0x98/0xa0 | ftracemodifyallcode+0xe0/0x144 | archftraceupdatecode+0x14/0x20 | ftracestartup+0xf8/0x1b0 | registerftracefunction+0x38/0x90 | testftraceinit+0xd0/0x1000 [testftrace] | dooneinitcall+0x50/0x2b0 | doinitmodule+0x50/0x1f0 | loadmodule+0x17c8/0x1d64 | _dosysfinitmodule+0xa8/0x100 | _arm64sysfinitmodule+0x2c/0x3c | invokesyscall+0x50/0x120 | el0svccommon.constprop.0+0xdc/0x100 | doel0svc+0x3c/0xd0 | el0svc+0x34/0xb0 | el0t64synchandler+0xbc/0x140 | el0t64sync+0x18c/0x190 | ---[ end trace 0000000000000000 ]---
We can solve this by consistently determining whether to use a PLT entry for an address.
Note that since (the earlier) commit:
f1a54ae9 ---truncated---
- References
-
- https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
- https://git.kernel.org/stable/c/a6253579977e4c6f7818eeb05bf2bc65678a7187
- https://git.kernel.org/stable/c/bc28fde90937a920f7714ec4408269cac744f796
- https://git.kernel.org/stable/c/db73aa9466338ec821ed2a0b01721fe4d06876b1
- https://git.kernel.org/stable/c/dcecc96ed16f73417de5550f384e348c9d56f279
Affected packages
Git / git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
Affected ranges
- Type
- GIT
- Repo
- https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
- Events
-
Introduced3b23e4991fb66f6d152f9055ede271a726ef9f21Fixedbc28fde90937a920f7714ec4408269cac744f796
- Type
- GIT
- Repo
- https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
- Events
-
Introduced3b23e4991fb66f6d152f9055ede271a726ef9f21Fixeddb73aa9466338ec821ed2a0b01721fe4d06876b1
- Type
- GIT
- Repo
- https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
- Events
-
Introduced3b23e4991fb66f6d152f9055ede271a726ef9f21Fixeddcecc96ed16f73417de5550f384e348c9d56f279
- Type
- GIT
- Repo
- https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
- Events
-
Introduced3b23e4991fb66f6d152f9055ede271a726ef9f21Fixeda6253579977e4c6f7818eeb05bf2bc65678a7187