CVE-2024-43837
Publication date:
17/08/2024
In the Linux kernel, the following vulnerability has been resolved:<br />
<br />
bpf: Fix null pointer dereference in resolve_prog_type() for BPF_PROG_TYPE_EXT<br />
<br />
When loading a EXT program without specifying `attr->attach_prog_fd`,<br />
the `prog->aux->dst_prog` will be null. At this time, calling<br />
resolve_prog_type() anywhere will result in a null pointer dereference.<br />
<br />
Example stack trace:<br />
<br />
[ 8.107863] Unable to handle kernel NULL pointer dereference at virtual address 0000000000000004<br />
[ 8.108262] Mem abort info:<br />
[ 8.108384] ESR = 0x0000000096000004<br />
[ 8.108547] EC = 0x25: DABT (current EL), IL = 32 bits<br />
[ 8.108722] SET = 0, FnV = 0<br />
[ 8.108827] EA = 0, S1PTW = 0<br />
[ 8.108939] FSC = 0x04: level 0 translation fault<br />
[ 8.109102] Data abort info:<br />
[ 8.109203] ISV = 0, ISS = 0x00000004, ISS2 = 0x00000000<br />
[ 8.109399] CM = 0, WnR = 0, TnD = 0, TagAccess = 0<br />
[ 8.109614] GCS = 0, Overlay = 0, DirtyBit = 0, Xs = 0<br />
[ 8.109836] user pgtable: 4k pages, 48-bit VAs, pgdp=0000000101354000<br />
[ 8.110011] [0000000000000004] pgd=0000000000000000, p4d=0000000000000000<br />
[ 8.112624] Internal error: Oops: 0000000096000004 [#1] PREEMPT SMP<br />
[ 8.112783] Modules linked in:<br />
[ 8.113120] CPU: 0 PID: 99 Comm: may_access_dire Not tainted 6.10.0-rc3-next-20240613-dirty #1<br />
[ 8.113230] Hardware name: linux,dummy-virt (DT)<br />
[ 8.113390] pstate: 60000005 (nZCv daif -PAN -UAO -TCO -DIT -SSBS BTYPE=--)<br />
[ 8.113429] pc : may_access_direct_pkt_data+0x24/0xa0<br />
[ 8.113746] lr : add_subprog_and_kfunc+0x634/0x8e8<br />
[ 8.113798] sp : ffff80008283b9f0<br />
[ 8.113813] x29: ffff80008283b9f0 x28: ffff800082795048 x27: 0000000000000001<br />
[ 8.113881] x26: ffff0000c0bb2600 x25: 0000000000000000 x24: 0000000000000000<br />
[ 8.113897] x23: ffff0000c1134000 x22: 000000000001864f x21: ffff0000c1138000<br />
[ 8.113912] x20: 0000000000000001 x19: ffff0000c12b8000 x18: ffffffffffffffff<br />
[ 8.113929] x17: 0000000000000000 x16: 0000000000000000 x15: 0720072007200720<br />
[ 8.113944] x14: 0720072007200720 x13: 0720072007200720 x12: 0720072007200720<br />
[ 8.113958] x11: 0720072007200720 x10: 0000000000f9fca4 x9 : ffff80008021f4e4<br />
[ 8.113991] x8 : 0101010101010101 x7 : 746f72705f6d656d x6 : 000000001e0e0f5f<br />
[ 8.114006] x5 : 000000000001864f x4 : ffff0000c12b8000 x3 : 000000000000001c<br />
[ 8.114020] x2 : 0000000000000002 x1 : 0000000000000000 x0 : 0000000000000000<br />
[ 8.114126] Call trace:<br />
[ 8.114159] may_access_direct_pkt_data+0x24/0xa0<br />
[ 8.114202] bpf_check+0x3bc/0x28c0<br />
[ 8.114214] bpf_prog_load+0x658/0xa58<br />
[ 8.114227] __sys_bpf+0xc50/0x2250<br />
[ 8.114240] __arm64_sys_bpf+0x28/0x40<br />
[ 8.114254] invoke_syscall.constprop.0+0x54/0xf0<br />
[ 8.114273] do_el0_svc+0x4c/0xd8<br />
[ 8.114289] el0_svc+0x3c/0x140<br />
[ 8.114305] el0t_64_sync_handler+0x134/0x150<br />
[ 8.114331] el0t_64_sync+0x168/0x170<br />
[ 8.114477] Code: 7100707f 54000081 f9401c00 f9403800 (b9400403)<br />
[ 8.118672] ---[ end trace 0000000000000000 ]---<br />
<br />
One way to fix it is by forcing `attach_prog_fd` non-empty when<br />
bpf_prog_load(). But this will lead to `libbpf_probe_bpf_prog_type`<br />
API broken which use verifier log to probe prog type and will log<br />
nothing if we reject invalid EXT prog before bpf_check().<br />
<br />
Another way is by adding null check in resolve_prog_type().<br />
<br />
The issue was introduced by commit 4a9c7bbe2ed4 ("bpf: Resolve to<br />
prog->aux->dst_prog->type only for BPF_PROG_TYPE_EXT") which wanted<br />
to correct type resolution for BPF_PROG_TYPE_TRACING programs. Before<br />
that, the type resolution of BPF_PROG_TYPE_EXT prog actually follows<br />
the logic below:<br />
<br />
prog->aux->dst_prog ? prog->aux->dst_prog->type : prog->type;<br />
<br />
It implies that when EXT program is not yet attached to `dst_prog`,<br />
the prog type should be EXT itself. This code worked fine in the past.<br />
So just keep using it.<br />
<br />
Fix this by returning `prog->type` for BPF_PROG_TYPE_EXT if `dst_prog`<br />
is not present in resolve_prog_type().
Severity CVSS v4.0: Pending analysis
Last modification:
03/11/2025