Yang Jihong
8569b4ada1
ftrace: Fix NULL pointer dereference in is_ftrace_trampoline when ftrace is dead
commit c3b0f72e805f0801f05fa2aa52011c4bfc694c44 upstream.
ftrace_startup does not remove ops from ftrace_ops_list when
ftrace_startup_enable fails:
register_ftrace_function
ftrace_startup
__register_ftrace_function
...
add_ftrace_ops(&ftrace_ops_list, ops)
...
...
ftrace_startup_enable // if ftrace failed to modify, ftrace_disabled is set to 1
...
return 0 // ops is in the ftrace_ops_list.
When ftrace_disabled = 1, unregister_ftrace_function simply returns without doing anything:
unregister_ftrace_function
ftrace_shutdown
if (unlikely(ftrace_disabled))
return -ENODEV; // return here, __unregister_ftrace_function is not executed,
// as a result, ops is still in the ftrace_ops_list
__unregister_ftrace_function
...
If ops is dynamically allocated, it will be free later, in this case,
is_ftrace_trampoline accesses NULL pointer:
is_ftrace_trampoline
ftrace_ops_trampoline
do_for_each_ftrace_op(op, ftrace_ops_list) // OOPS! op may be NULL!
Syzkaller reports as follows:
[ 1203.506103] BUG: kernel NULL pointer dereference, address: 000000000000010b
[ 1203.508039] #PF: supervisor read access in kernel mode
[ 1203.508798] #PF: error_code(0x0000) - not-present page
[ 1203.509558] PGD 800000011660b067 P4D 800000011660b067 PUD 130fb8067 PMD 0
[ 1203.510560] Oops: 0000 [#1] SMP KASAN PTI
[ 1203.511189] CPU: 6 PID: 29532 Comm: syz-executor.2 Tainted: G B W 5.10.0 #8
[ 1203.512324] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS rel-1.14.0-0-g155821a1990b-prebuilt.qemu.org 04/01/2014
[ 1203.513895] RIP: 0010:is_ftrace_trampoline+0x26/0xb0
[ 1203.514644] Code: ff eb d3 90 41 55 41 54 49 89 fc 55 53 e8 f2 00 fd ff 48 8b 1d 3b 35 5d 03 e8 e6 00 fd ff 48 8d bb 90 00 00 00 e8 2a 81 26 00 <48> 8b ab 90 00 00 00 48 85 ed 74 1d e8 c9 00 fd ff 48 8d bb 98 00
[ 1203.518838] RSP: 0018:ffffc900012cf960 EFLAGS: 00010246
[ 1203.520092] RAX: 0000000000000000 RBX: 000000000000007b RCX: ffffffff8a331866
[ 1203.521469] RDX: 0000000000000000 RSI: 0000000000000008 RDI: 000000000000010b
[ 1203.522583] RBP: 0000000000000000 R08: 0000000000000000 R09: ffffffff8df18b07
[ 1203.523550] R10: fffffbfff1be3160 R11: 0000000000000001 R12: 0000000000478399
[ 1203.524596] R13: 0000000000000000 R14: ffff888145088000 R15: 0000000000000008
[ 1203.525634] FS: 00007f429f5f4700(0000) GS:ffff8881daf00000(0000) knlGS:0000000000000000
[ 1203.526801] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
[ 1203.527626] CR2: 000000000000010b CR3: 0000000170e1e001 CR4: 00000000003706e0
[ 1203.528611] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
[ 1203.529605] DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7: 0000000000000400
Therefore, when ftrace_startup_enable fails, we need to rollback registration
process and remove ops from ftrace_ops_list.
Link: https://lkml.kernel.org/r/20220818032659.56209-1-yangjihong1@huawei.com
Suggested-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Yang Jihong <yangjihong1@huawei.com>
Signed-off-by: Steven Rostedt (Google) <rostedt@goodmis.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2022-09-05 10:23:57 +02:00
..
2022-08-25 11:09:24 +02:00
2016-10-11 15:06:32 -07:00
2021-03-03 17:44:31 +01:00
2022-07-29 17:05:45 +02:00
2020-09-23 08:46:14 +02:00
2022-06-25 11:45:04 +02:00
2016-08-26 14:42:08 +02:00
2021-11-26 11:48:27 +01:00
2022-04-20 09:06:31 +02:00
2022-04-20 09:06:33 +02:00
2019-05-31 06:48:30 -07:00
2022-04-20 09:06:32 +02:00
2022-06-25 11:45:13 +02:00
2022-09-05 10:23:57 +02:00
2018-01-10 09:29:51 +01:00
2018-02-17 13:21:18 +01:00
2020-10-01 20:40:07 +02:00
2021-04-07 12:05:41 +02:00
2021-04-07 12:05:41 +02:00
2021-04-07 12:05:41 +02:00
2019-11-28 18:28:55 +01:00
2018-11-13 11:16:57 -08:00
2017-01-06 10:40:13 +01:00
2019-12-21 10:42:02 +01:00
2022-02-11 08:41:17 +01:00
2020-06-20 10:24:21 +02:00
2022-06-25 11:45:10 +02:00
2017-10-12 11:51:25 +02:00
2020-01-12 11:24:13 +01:00
2022-07-29 17:05:44 +02:00
2017-07-21 07:42:21 +02:00
2021-09-22 11:43:08 +02:00
2016-07-28 16:07:41 -07:00
2021-07-11 12:46:40 +02:00
2018-01-10 09:29:52 +01:00
2019-04-20 09:07:52 +02:00
2017-12-14 09:28:24 +01:00
2018-08-03 07:55:12 +02:00
2018-06-05 10:28:57 +02:00
2022-07-02 16:17:16 +02:00
2016-08-02 19:35:27 -04:00
2020-10-14 09:48:14 +02:00
2021-12-08 08:45:06 +01:00
2016-08-02 19:35:30 -04:00
2021-07-11 12:46:41 +02:00
2021-02-10 09:09:25 +01:00
2017-03-12 06:41:45 +01:00
2019-01-13 10:03:51 +01:00
2021-03-03 17:44:44 +01:00
2020-04-02 17:20:26 +02:00
2020-05-27 16:41:53 +02:00
2019-10-17 13:42:25 -07:00
2021-09-22 11:43:08 +02:00
2018-04-13 19:47:53 +02:00
2021-09-26 13:36:18 +02:00
2022-06-14 16:52:28 +02:00
2020-11-18 18:26:32 +01:00
2020-08-26 10:29:03 +02:00
2018-04-24 09:34:09 +02:00
2021-03-03 17:44:43 +01:00
2022-07-21 20:40:32 +02:00
2022-04-20 09:06:46 +02:00
2016-10-11 15:06:33 -07:00
2018-08-15 18:14:42 +02:00
2017-02-14 15:25:42 -08:00
2018-08-03 07:55:24 +02:00
2016-09-13 14:41:36 +02:00
2021-09-26 13:36:18 +02:00
2022-03-11 10:03:31 +01:00
2016-08-02 19:35:02 -04:00
2020-01-12 11:24:12 +01:00
2016-08-22 10:01:49 -07:00
2021-03-03 17:44:38 +01:00
2022-02-23 11:56:40 +01:00
2017-06-17 06:41:51 +02:00
2018-01-10 09:29:52 +01:00
2016-09-05 13:52:39 +02:00
2018-09-09 20:01:24 +02:00
2018-09-09 20:01:24 +02:00
2016-09-22 20:00:36 -05:00
2017-06-17 06:41:57 +02:00
2017-06-17 06:41:57 +02:00
2017-11-15 15:53:17 +01:00
2022-06-25 11:45:09 +02:00