lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <67078f7d.050a0220.1139e6.001e.GAE@google.com>
Date: Thu, 10 Oct 2024 01:25:33 -0700
From: syzbot <syzbot+5fe14f2ff4ccbace9a26@...kaller.appspotmail.com>
To: linux-kernel@...r.kernel.org, syzkaller-bugs@...glegroups.com
Subject: Re: [syzbot] Re: [syzbot] [net?] KASAN: slab-use-after-free Read in __ethtool_get_link_ksettings

For archival purposes, forwarding an incoming command email to
linux-kernel@...r.kernel.org, syzkaller-bugs@...glegroups.com.

***

Subject: Re: [syzbot] [net?] KASAN: slab-use-after-free Read in __ethtool_get_link_ksettings
Author: cmeiohas@...dia.com

#syz test: https://git.kernel.org/pub/scm/linux/kernel/git/rdma/rdma.git for-next

diff --git a/drivers/infiniband/core/device.c b/drivers/infiniband/core/device.c
index 93c6d27b5d8f..eafdf1741582 100644
--- a/drivers/infiniband/core/device.c
+++ b/drivers/infiniband/core/device.c
@@ -2062,19 +2062,14 @@ void ib_dispatch_event_clients(struct ib_event *event)
     up_read(&event->device->event_handler_rwsem);
 }

-static int iw_query_port(struct ib_device *device,
-               u32 port_num,
-               struct ib_port_attr *port_attr)
+static int iw_query_port(struct ib_device *device, u32 port_num,
+                        struct ib_port_attr *port_attr,
+                        struct net_device *netdev)
 {
     struct in_device *inetdev;
-    struct net_device *netdev;

     memset(port_attr, 0, sizeof(*port_attr));

-    netdev = ib_device_get_netdev(device, port_num);
-    if (!netdev)
-        return -ENODEV;
-
     port_attr->max_mtu = IB_MTU_4096;
     port_attr->active_mtu = ib_mtu_int_to_enum(netdev->mtu);

@@ -2097,7 +2092,6 @@ static int iw_query_port(struct ib_device *device,
         rcu_read_unlock();
     }

-    dev_put(netdev);
     return device->ops.query_port(device, port_num, port_attr);
 }

@@ -2135,13 +2129,27 @@ int ib_query_port(struct ib_device *device,
           u32 port_num,
           struct ib_port_attr *port_attr)
 {
+       struct net_device *netdev = NULL;
+       int ret;
+
     if (!rdma_is_port_valid(device, port_num))
         return -EINVAL;

+       if (rdma_protocol_iwarp(device, port_num) ||
+           rdma_protocol_roce(device, port_num)) {
+               netdev = ib_device_get_netdev(device, port_num);
+               if (!netdev)
+                       return -ENODEV;
+       }
+
     if (rdma_protocol_iwarp(device, port_num))
-        return iw_query_port(device, port_num, port_attr);
+               ret = iw_query_port(device, port_num, port_attr, netdev);
     else
-        return __ib_query_port(device, port_num, port_attr);
+               ret = __ib_query_port(device, port_num, port_attr);
+       if (netdev)
+               dev_put(netdev);
+       return ret;
+
 }
 EXPORT_SYMBOL(ib_query_port);

diff --git a/drivers/infiniband/sw/rxe/rxe_net.c b/drivers/infiniband/sw/rxe/rxe_net.c
index 75d1407db52d..c5d439bebc24 100644
--- a/drivers/infiniband/sw/rxe/rxe_net.c
+++ b/drivers/infiniband/sw/rxe/rxe_net.c
@@ -605,6 +605,7 @@ static int rxe_notify(struct notifier_block *not_blk,

     switch (event) {
     case NETDEV_UNREGISTER:
+        ib_device_set_netdev(&rxe->ib_dev, NULL, 1);
         ib_unregister_device_queued(&rxe->ib_dev);
         break;
     case NETDEV_UP:
diff --git a/drivers/infiniband/sw/siw/siw_main.c b/drivers/infiniband/sw/siw/siw_main.c
index 17abef48abcd..1b7d34798783 100644
--- a/drivers/infiniband/sw/siw/siw_main.c
+++ b/drivers/infiniband/sw/siw/siw_main.c
@@ -400,6 +400,7 @@ static int siw_netdev_event(struct notifier_block *nb, unsigned long event,
         break;

     case NETDEV_UNREGISTER:
+        ib_device_set_netdev(&sdev->base_dev, NULL, 1);
         ib_unregister_device_queued(&sdev->base_dev);
         break;


On 08/10/2024 15:24, syzbot wrote:
> External email: Use caution opening links or attachments
>
>
> Hello,
>
> syzbot has tested the proposed patch but the reproducer is still triggering an issue:
> KASAN: slab-use-after-free Read in __ethtool_get_link_ksettings
>
> ==================================================================
> BUG: KASAN: slab-use-after-free in __ethtool_get_link_ksettings+0x6e/0x190 net/ethtool/ioctl.c:442
> Read of size 8 at addr ffff8880673be308 by task kworker/0:1/9
>
> CPU: 0 UID: 0 PID: 9 Comm: kworker/0:1 Not tainted 6.12.0-rc2-syzkaller-g615b94746a54-dirty #0
> Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS Google 09/13/2024
> Workqueue: events smc_ib_port_event_work
> Call Trace:
>  <TASK>
>  __dump_stack lib/dump_stack.c:94 [inline]
>  dump_stack_lvl+0x241/0x360 lib/dump_stack.c:120
>  print_address_description mm/kasan/report.c:377 [inline]
>  print_report+0x169/0x550 mm/kasan/report.c:488
>  kasan_report+0x143/0x180 mm/kasan/report.c:601
>  __ethtool_get_link_ksettings+0x6e/0x190 net/ethtool/ioctl.c:442
>  ib_get_eth_speed+0x160/0x800 drivers/infiniband/core/verbs.c:1996
>  rxe_query_port+0x76/0x260 drivers/infiniband/sw/rxe/rxe_verbs.c:55
>  __ib_query_port drivers/infiniband/core/device.c:2105 [inline]
>  ib_query_port+0x208/0x870 drivers/infiniband/core/device.c:2147
>  smc_ib_remember_port_attr net/smc/smc_ib.c:364 [inline]
>  smc_ib_port_event_work+0x14e/0xa50 net/smc/smc_ib.c:388
>  process_one_work kernel/workqueue.c:3229 [inline]
>  process_scheduled_works+0xa63/0x1850 kernel/workqueue.c:3310
>  worker_thread+0x870/0xd30 kernel/workqueue.c:3391
>  kthread+0x2f0/0x390 kernel/kthread.c:389
>  ret_from_fork+0x4b/0x80 arch/x86/kernel/process.c:147
>  ret_from_fork_asm+0x1a/0x30 arch/x86/entry/entry_64.S:244
>  </TASK>
>
> Allocated by task 5967:
>  kasan_save_stack mm/kasan/common.c:47 [inline]
>  kasan_save_track+0x3f/0x80 mm/kasan/common.c:68
>  poison_kmalloc_redzone mm/kasan/common.c:377 [inline]
>  __kasan_kmalloc+0x98/0xb0 mm/kasan/common.c:394
>  kasan_kmalloc include/linux/kasan.h:257 [inline]
>  __do_kmalloc_node mm/slub.c:4264 [inline]
>  __kmalloc_node_noprof+0x22a/0x440 mm/slub.c:4270
>  __kvmalloc_node_noprof+0x72/0x190 mm/util.c:658
>  alloc_netdev_mqs+0x9b/0x1000 net/core/dev.c:11097
>  rtnl_create_link+0x2f9/0xc20 net/core/rtnetlink.c:3374
>  rtnl_newlink_create net/core/rtnetlink.c:3500 [inline]
>  __rtnl_newlink net/core/rtnetlink.c:3730 [inline]
>  rtnl_newlink+0x1423/0x20a0 net/core/rtnetlink.c:3743
>  rtnetlink_rcv_msg+0x73f/0xcf0 net/core/rtnetlink.c:6646
>  netlink_rcv_skb+0x1e3/0x430 net/netlink/af_netlink.c:2550
>  netlink_unicast_kernel net/netlink/af_netlink.c:1331 [inline]
>  netlink_unicast+0x7f6/0x990 net/netlink/af_netlink.c:1357
>  netlink_sendmsg+0x8e4/0xcb0 net/netlink/af_netlink.c:1901
>  sock_sendmsg_nosec net/socket.c:729 [inline]
>  __sock_sendmsg+0x221/0x270 net/socket.c:744
>  __sys_sendto+0x39b/0x4f0 net/socket.c:2209
>  __do_sys_sendto net/socket.c:2221 [inline]
>  __se_sys_sendto net/socket.c:2217 [inline]
>  __x64_sys_sendto+0xde/0x100 net/socket.c:2217
>  do_syscall_x64 arch/x86/entry/common.c:52 [inline]
>  do_syscall_64+0xf3/0x230 arch/x86/entry/common.c:83
>  entry_SYSCALL_64_after_hwframe+0x77/0x7f
>
> Freed by task 11:
>  kasan_save_stack mm/kasan/common.c:47 [inline]
>  kasan_save_track+0x3f/0x80 mm/kasan/common.c:68
>  kasan_save_free_info+0x40/0x50 mm/kasan/generic.c:579
>  poison_slab_object mm/kasan/common.c:247 [inline]
>  __kasan_slab_free+0x59/0x70 mm/kasan/common.c:264
>  kasan_slab_free include/linux/kasan.h:230 [inline]
>  slab_free_hook mm/slub.c:2342 [inline]
>  slab_free mm/slub.c:4579 [inline]
>  kfree+0x1a0/0x440 mm/slub.c:4727
>  device_release+0x99/0x1c0
>  kobject_cleanup lib/kobject.c:689 [inline]
>  kobject_release lib/kobject.c:720 [inline]
>  kref_put include/linux/kref.h:65 [inline]
>  kobject_put+0x22f/0x480 lib/kobject.c:737
>  netdev_run_todo+0xe79/0x1000 net/core/dev.c:10816
>  default_device_exit_batch+0xa24/0xaa0 net/core/dev.c:11949
>  ops_exit_list net/core/net_namespace.c:178 [inline]
>  cleanup_net+0x89d/0xcc0 net/core/net_namespace.c:626
>  process_one_work kernel/workqueue.c:3229 [inline]
>  process_scheduled_works+0xa63/0x1850 kernel/workqueue.c:3310
>  worker_thread+0x870/0xd30 kernel/workqueue.c:3391
>  kthread+0x2f0/0x390 kernel/kthread.c:389
>  ret_from_fork+0x4b/0x80 arch/x86/kernel/process.c:147
>  ret_from_fork_asm+0x1a/0x30 arch/x86/entry/entry_64.S:244
>
> The buggy address belongs to the object at ffff8880673be000
>  which belongs to the cache kmalloc-cg-4k of size 4096
> The buggy address is located 776 bytes inside of
>  freed 4096-byte region [ffff8880673be000, ffff8880673bf000)
>
> The buggy address belongs to the physical page:
> page: refcount:1 mapcount:0 mapping:0000000000000000 index:0x0 pfn:0x673b8
> head: order:3 mapcount:0 entire_mapcount:0 nr_pages_mapped:0 pincount:0
> memcg:ffff888031406a41
> flags: 0xfff00000000040(head|node=0|zone=1|lastcpupid=0x7ff)
> page_type: f5(slab)
> raw: 00fff00000000040 ffff88801ac4f500 dead000000000122 0000000000000000
> raw: 0000000000000000 0000000000040004 00000001f5000000 ffff888031406a41
> head: 00fff00000000040 ffff88801ac4f500 dead000000000122 0000000000000000
> head: 0000000000000000 0000000000040004 00000001f5000000 ffff888031406a41
> head: 00fff00000000003 ffffea00019cee01 ffffffffffffffff 0000000000000000
> head: 0000000700000008 0000000000000000 00000000ffffffff 0000000000000000
> page dumped because: kasan: bad access detected
> page_owner tracks the page as allocated
> page last allocated via order 3, migratetype Unmovable, gfp_mask 0xd20c0(__GFP_IO|__GFP_FS|__GFP_NOWARN|__GFP_NORETRY|__GFP_COMP|__GFP_NOMEMALLOC), pid 5982, tgid 5982 (syz-executor), ts 102124010087, free_ts 96435310364
>  set_page_owner include/linux/page_owner.h:32 [inline]
>  post_alloc_hook+0x1f3/0x230 mm/page_alloc.c:1537
>  prep_new_page mm/page_alloc.c:1545 [inline]
>  get_page_from_freelist+0x3045/0x3190 mm/page_alloc.c:3457
>  __alloc_pages_noprof+0x256/0x6c0 mm/page_alloc.c:4733
>  alloc_pages_mpol_noprof+0x3e8/0x680 mm/mempolicy.c:2265
>  alloc_slab_page+0x6a/0x120 mm/slub.c:2412
>  allocate_slab+0x5a/0x2f0 mm/slub.c:2578
>  new_slab mm/slub.c:2631 [inline]
>  ___slab_alloc+0xcd1/0x14b0 mm/slub.c:3818
>  __slab_alloc+0x58/0xa0 mm/slub.c:3908
>  __slab_alloc_node mm/slub.c:3961 [inline]
>  slab_alloc_node mm/slub.c:4122 [inline]
>  __kmalloc_cache_noprof+0x1d5/0x2c0 mm/slub.c:4290
>  kmalloc_noprof include/linux/slab.h:878 [inline]
>  kzalloc_noprof include/linux/slab.h:1014 [inline]
>  snmp6_alloc_dev net/ipv6/addrconf.c:359 [inline]
>  ipv6_add_dev+0x570/0x1220 net/ipv6/addrconf.c:409
>  addrconf_notify+0x6a7/0x1020 net/ipv6/addrconf.c:3655
>  notifier_call_chain+0x19f/0x3e0 kernel/notifier.c:93
>  call_netdevice_notifiers_extack net/core/dev.c:2034 [inline]
>  call_netdevice_notifiers net/core/dev.c:2048 [inline]
>  register_netdevice+0x167f/0x1b00 net/core/dev.c:10524
>  veth_newlink+0x855/0xce0 drivers/net/veth.c:1861
>  rtnl_newlink_create net/core/rtnetlink.c:3510 [inline]
>  __rtnl_newlink net/core/rtnetlink.c:3730 [inline]
>  rtnl_newlink+0x1591/0x20a0 net/core/rtnetlink.c:3743
>  rtnetlink_rcv_msg+0x73f/0xcf0 net/core/rtnetlink.c:6646
> page last free pid 5869 tgid 5869 stack trace:
>  reset_page_owner include/linux/page_owner.h:25 [inline]
>  free_pages_prepare mm/page_alloc.c:1108 [inline]
>  free_unref_folios+0xf12/0x18d0 mm/page_alloc.c:2686
>  folios_put_refs+0x76c/0x860 mm/swap.c:1007
>  free_pages_and_swap_cache+0x2ea/0x690 mm/swap_state.c:332
>  __tlb_batch_free_encoded_pages mm/mmu_gather.c:136 [inline]
>  tlb_batch_pages_flush mm/mmu_gather.c:149 [inline]
>  tlb_flush_mmu_free mm/mmu_gather.c:366 [inline]
>  tlb_flush_mmu+0x3a3/0x680 mm/mmu_gather.c:373
>  tlb_finish_mmu+0xd4/0x200 mm/mmu_gather.c:465
>  vms_clear_ptes+0x437/0x530 mm/vma.c:1096
>  vms_complete_munmap_vmas+0x208/0x910 mm/vma.c:1140
>  do_vmi_align_munmap+0x613/0x730 mm/vma.c:1349
>  do_vmi_munmap+0x24e/0x2d0 mm/vma.c:1397
>  __vm_munmap+0x24c/0x480 mm/mmap.c:1600
>  __do_sys_munmap mm/mmap.c:1617 [inline]
>  __se_sys_munmap mm/mmap.c:1614 [inline]
>  __x64_sys_munmap+0x68/0x80 mm/mmap.c:1614
>  do_syscall_x64 arch/x86/entry/common.c:52 [inline]
>  do_syscall_64+0xf3/0x230 arch/x86/entry/common.c:83
>  entry_SYSCALL_64_after_hwframe+0x77/0x7f
>
> Memory state around the buggy address:
>  ffff8880673be200: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
>  ffff8880673be280: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
>> ffff8880673be300: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
>                       ^
>  ffff8880673be380: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
>  ffff8880673be400: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb
> ==================================================================
>
>
> Tested on:
>
> commit:         615b9474 RDMA/hns: Disassociate mmap pages for all uct..
> git tree:       https://git.kernel.org/pub/scm/linux/kernel/git/rdma/rdma.git for-next
> console output: https://syzkaller.appspot.com/x/log.txt?x=131b1707980000
> kernel config:  https://syzkaller.appspot.com/x/.config?x=7cd9e7e4a8a0a15b
> dashboard link: https://syzkaller.appspot.com/bug?extid=5fe14f2ff4ccbace9a26
> compiler:       Debian clang version 15.0.6, GNU ld (GNU Binutils for Debian) 2.40
> patch:          https://syzkaller.appspot.com/x/patch.diff?x=17698f9f980000
>
>

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ