CVE: CVE-2021-47274
Published: 2024-05-21
Description: In the Linux kernel, the following vulnerability has been resolved:

tracing: Correct the length check which causes memory corruption

We've suffered from severe kernel crashes due to memory corruption on
our production environment, like,

Call Trace:
[[PII_REDACTED]77] general protection fault: 0000 [#1] SMP PTI
[[PII_REDACTED]56] CPU: 17 PID: 26996 Comm: python Kdump: loaded Tainted:G
[[PII_REDACTED]29] RIP: 0010:kmem_cache_alloc+0x90/0x190
[[PII_REDACTED]74] RSP: 0018:ffffb16faa597df8 EFLAGS: 00010286
[[PII_REDACTED]87] RAX: [PII_REDACTED] RBX: [PII_REDACTED] RCX:[PII_REDACTED]e931bf
[[PII_REDACTED]23] RDX:[PII_REDACTED]e931be RSI: [PII_REDACTED] RDI:
ffff9a45ff004300
[[PII_REDACTED]96] RBP: [PII_REDACTED] R08: [PII_REDACTED] R09:
[PII_REDACTED]
[[PII_REDACTED]70] R10: [PII_REDACTED] R11: [PII_REDACTED] R12:
ffffffff9a20608d
[[PII_REDACTED]66] R13: ffff9a45ff004300 R14: ffff9a45ff004300 R15:
696c662f65636976
[[PII_REDACTED]28] FS:  00007f45d7c6f740(0000) GS:ffff9a45ff840000(0000)
knlGS:[PII_REDACTED]
[[PII_REDACTED]37] CS:  0010 DS: 0000 ES: 0000 CR0: [PII_REDACTED]
[[PII_REDACTED]57] CR2: 00007f45d71311a0 CR3: 000000189d63e004 CR4:[PII_REDACTED]3606e0
[[PII_REDACTED]79] DR0: [PII_REDACTED] DR1: [PII_REDACTED] DR2:
[PII_REDACTED]
[[PII_REDACTED]69] DR3: [PII_REDACTED] DR6: 00000000fffe0ff0 DR7:
[PII_REDACTED]
[[PII_REDACTED]42] Call Trace:
[[PII_REDACTED]09]  anon_vma_clone+0x5d/0x170
[[PII_REDACTED]17]  __split_vma+0x91/0x1a0
[[PII_REDACTED]77]  do_munmap+0x2c6/0x320
[[PII_REDACTED]28]  vm_munmap+0x54/0x70
[[PII_REDACTED]90]  __x64_sys_munmap+0x22/0x30
[[PII_REDACTED]05]  do_syscall_64+0x5b/0x1b0
[[PII_REDACTED]24]  entry_SYSCALL_64_after_hwframe+0x44/0xa9
[[PII_REDACTED]42] RIP: 0033:0x7f45d6e61e27

James Wang has reproduced it stably on the latest 4.19 LTS.
After some debugging, we finally proved that it's due to ftrace
buffer out-of-bound access using a debug tool as follows:
[   86.775200] BUG: Out-of-bounds write at addr 0xffff88aefe8b7000
[   86.780806]  no_context+0xdf/0x3c0
[   86.784327]  __do_page_fault+0x252/0x470
[   86.788367]  do_page_fault+0x32/0x140
[   86.792145]  page_fault+0x1e/0x30
[   86.795576]  strncpy_from_unsafe+0x66/0xb0
[   86.799789]  fetch_memory_string+0x25/0x40
[   86.804002]  fetch_deref_string+0x51/0x60
[   86.808134]  kprobe_trace_func+0x32d/0x3a0
[   86.812347]  kprobe_dispatcher+0x45/0x50
[   86.816385]  kprobe_ftrace_handler+0x90/0xf0
[   86.820779]  ftrace_ops_assist_func+0xa1/0x140
[   86.825340]  0xffffffffc00750bf
[   86.828603]  do_sys_open+0x5/0x1f0
[   86.832124]  do_syscall_64+0x5b/0x1b0
[   86.835900]  entry_SYSCALL_64_after_hwframe+0x44/0xa9

commit b220c049d519 ("tracing: Check length before giving out
the filter buffer") adds length check to protect trace data
overflow introduced in 0fc1b09ff1ff, seems that this fix can't prevent
overflow entirely, the length check should also take the sizeof
entry->array[0] into account, since this array[0] is filled the
length of trace data and occupy addtional space and risk overflow.
CVSS Score: 9.8 (CRITICAL)
CWE: CWE-125
References: https://git.kernel.org/stable/c/2d[PII_REDACTED]86d67947406f26ee8e5fd2ca097f; https://git.kernel.org/stable/c/31ceae385556c37e4d286cb[PII_REDACTED]f566883; https://git.kernel.org/stable/c/3e08a9f9760f4a70d633c328a76408e62d6f80a3