* [PATCH linux-next 0/4] fix possile NULL pointer dereference in ndo_get_iflink callback functions @ 2015-04-14 15:20 ` Honggang Li 0 siblings, 0 replies; 53+ messages in thread From: Honggang Li @ 2015-04-14 15:20 UTC (permalink / raw) To: roland-DgEjT+Ai2ygdnm+yROfE0A, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, kaber-dcUjhNyLwpNeoWH0uzbU5w, davem-fT/PcQaiUtIeIZ0/mPfg9Q, alex.estrin-ral2JQCrhuEAvxtiuMwx3w, dledford-H+wXaHxf7aLQT0dZR+AlfA, edumazet-hpIqsD4AKlfQT0dZR+AlfA, erezsh-VPRAkNaXOzVWk0Htik3J/w, nicolas.dichtel-pdR9zngts4EAvxtiuMwx3w, maheshb-hpIqsD4AKlfQT0dZR+AlfA, jbenc-H+wXaHxf7aLQT0dZR+AlfA, ebiederm-aS9lmoZGLiVWk0Htik3J/w, elfring-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f, f.fainelli-Re5JQEeQqe8AvxtiuMwx3w, linux-0h96xk9xTtrk1uMJSBkQmQ, andrew-g2DYL2Zd6BY, sfeldma-Re5JQEeQqe8AvxtiuMwx3w, alexander.h.duyck-ral2JQCrhuEAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA Cc: Honggang Li The four callback functions deferference the pointers without test. As ipoib_get_iflink failed, it is a good reason to fix the other three functions. Those functions will return zero on error, as valid ifindex greater than zero. I understand that those simple patches should be emerged into a single patch. However, those files have be maintained by different maintainers. So, I split it for review. It also will be convenient to remove one or more patches with the rest not be affected. Honggang Li (4): infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink ipvlan: fix possible NULL pointer dereference in ipvlan_get_iflink macvlan: fix possible NULL pointer dereference in macvlan_dev_get_iflink net/dsa: fix possible NULL pointer dereference in dsa_slave_get_iflink drivers/infiniband/ulp/ipoib/ipoib_main.c | 5 ++++- drivers/net/ipvlan/ipvlan_main.c | 5 ++++- drivers/net/macvlan.c | 4 +++- net/dsa/slave.c | 5 ++++- 4 files changed, 15 insertions(+), 4 deletions(-) -- 1.8.3.1 -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 53+ messages in thread
* [PATCH linux-next 0/4] fix possile NULL pointer dereference in ndo_get_iflink callback functions @ 2015-04-14 15:20 ` Honggang Li 0 siblings, 0 replies; 53+ messages in thread From: Honggang Li @ 2015-04-14 15:20 UTC (permalink / raw) To: roland, sean.hefty, hal.rosenstock, kaber, davem, alex.estrin, dledford, edumazet, erezsh, nicolas.dichtel, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev Cc: Honggang Li The four callback functions deferference the pointers without test. As ipoib_get_iflink failed, it is a good reason to fix the other three functions. Those functions will return zero on error, as valid ifindex greater than zero. I understand that those simple patches should be emerged into a single patch. However, those files have be maintained by different maintainers. So, I split it for review. It also will be convenient to remove one or more patches with the rest not be affected. Honggang Li (4): infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink ipvlan: fix possible NULL pointer dereference in ipvlan_get_iflink macvlan: fix possible NULL pointer dereference in macvlan_dev_get_iflink net/dsa: fix possible NULL pointer dereference in dsa_slave_get_iflink drivers/infiniband/ulp/ipoib/ipoib_main.c | 5 ++++- drivers/net/ipvlan/ipvlan_main.c | 5 ++++- drivers/net/macvlan.c | 4 +++- net/dsa/slave.c | 5 ++++- 4 files changed, 15 insertions(+), 4 deletions(-) -- 1.8.3.1 ^ permalink raw reply [flat|nested] 53+ messages in thread
* [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink 2015-04-14 15:20 ` Honggang Li (?) @ 2015-04-14 15:20 ` Honggang Li [not found] ` <1429024817-21561-2-git-send-email-honli-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2015-04-14 16:30 ` Erez Shitrit -1 siblings, 2 replies; 53+ messages in thread From: Honggang Li @ 2015-04-14 15:20 UTC (permalink / raw) To: roland, sean.hefty, hal.rosenstock, kaber, davem, alex.estrin, dledford, edumazet, erezsh, nicolas.dichtel, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev Cc: Honggang Li Starting monitoring for VG vg_rdma01: 3 logical volume(s) in volume group "vg_rdma01" monitored [ OK ] Starting cgconfig service: Failed to parse /etc/cgconfig.conf or /etc/cgconfig.d[FAILED] Loading OpenIB kernel modules: BUG: unable to handle kernel NULL pointer dereference at 0000000000000120 IP: [<ffffffffa06b9060>] ipoib_get_iflink+0x10/0x20 [ib_ipoib] PGD 475540067 PUD 473541067 PMD 0 Oops: 0000 [#1] SMP Modules linked in: ib_ipoib(+) rdma_ucm ib_ucm ib_uverbs ib_umad rdma_cm ib_cm ib_sa vhost_net macvtap macvlan vhost tun ipmi_devintf sg ipmi_si ipmi_msghandler serio_raw iTCO_wdt iTCO_vendor_support cdc_ether usbnet mii bnx2 intel_powerclamp coretemp kvm_intel kvm crc32c_intel ghash_clmulni_intel aesni_intel ablk_helper cryptd lrw gf128mul glue_helper aes_x86_64 microcode pcspkr i2c_i801 i2c_core lpc_ich mfd_core acpi_cpufreq ioatdma i7core_edac edac_core shpchp ext4(E) jbd2(E) mbcache(E) sd_mod(E) megaraid_sas(E) pata_acpi(E) ata_generic(E) ata_piix(E) iw_cxgb3(E) cxgb3(E) mdio(E) ib_qib(E) dca(E) ib_mad(E) iw_cxgb4(E) iw_cm(E) ib_core(E) ib_addr(E) ipv6(E) cxgb4(E) dm_mirror(E) dm_region_hash(E) dm_log(E) dm_mod(E) CPU: 6 PID: 2405 Comm: modprobe Tainted: G E 4.0.0-next-20150413 #1 Hardware name: IBM System x3650 M3 -[7945O63]-/00D4062, BIOS -[D6E157AUS-1.15]- 06/13/2012 task: ffff880476ad6f00 ti: ffff88047579c000 task.ti: ffff88047579c000 RIP: 0010:[<ffffffffa06b9060>] [<ffffffffa06b9060>] ipoib_get_iflink+0x10/0x20 [ib_ipoib] RSP: 0018:ffff88047579f9b8 EFLAGS: 00010286 RAX: 0000000000000000 RBX: ffff880476e2a000 RCX: 0000000000000000 RDX: 0000000000000004 RSI: ffff88047579fbb8 RDI: ffff880476e2a000 RBP: ffff88047579f9b8 R08: 0000000000000660 R09: ffff88047404f068 R10: 0000000000000000 R11: 0000000000000000 R12: ffff8804736bec00 R13: ffff88047579fbb4 R14: ffff88047404f000 R15: 0000000000000009 FS: 00007fc047a2e700(0000) GS:ffff88047fc00000(0000) knlGS:0000000000000000 CS: 0010 DS: 0000 ES: 0000 CR0: 000000008005003b CR2: 0000000000000120 CR3: 000000047541f000 CR4: 00000000000006e0 Stack: ffff88047579f9c8 ffffffff814fbfa3 ffff88047579fbe8 ffffffff81515a15 0000000000000005 ffff880476e2a280 0000000000000005 0000000000000014 ffff88047579fa48 ffffffff8150a577 0000000000000000 ffff8804ffffffff Call Trace: [<ffffffff814fbfa3>] dev_get_iflink+0x23/0x40 [<ffffffff81515a15>] rtnl_fill_ifinfo+0x255/0xce0 [<ffffffff8150a577>] ? __hw_addr_create_ex+0x97/0xc0 [<ffffffff815d32bb>] ? _raw_spin_unlock_bh+0x1b/0x20 [<ffffffff8150a8e5>] ? __dev_mc_add+0x75/0x90 [<ffffffffa00a115c>] ? igmp6_group_added+0x5c/0x130 [ipv6] [<ffffffff8119c6cc>] ? __kmalloc_node_track_caller+0x3c/0x50 [<ffffffff814f0f0b>] ? __kmalloc_reserve+0x3b/0xa0 [<ffffffff814f12f8>] ? __alloc_skb+0xa8/0x1f0 [<ffffffff81516783>] rtmsg_ifinfo_build_skb+0x83/0xe0 [<ffffffff81078fa6>] ? raw_notifier_call_chain+0x16/0x20 [<ffffffff81516801>] rtmsg_ifinfo+0x21/0x40 [<ffffffff81504eaf>] register_netdevice+0x38f/0x400 [<ffffffff81504f3e>] register_netdev+0x1e/0x30 [<ffffffffa06bc204>] ipoib_add_port.clone.0+0x214/0x390 [ib_ipoib] [<ffffffffa06bc447>] ipoib_add_one+0xc7/0x110 [ib_ipoib] [<ffffffffa00f9d4d>] ib_register_client+0x7d/0xa0 [ib_core] [<ffffffffa06ce000>] ? 0xffffffffa06ce000 [<ffffffffa06ce0f2>] ipoib_init_module+0xf2/0x13c [ib_ipoib] [<ffffffff81000287>] do_one_initcall+0xb7/0x1d0 [<ffffffff810d8189>] do_init_module+0x69/0x200 [<ffffffff810da985>] load_module+0x5b5/0x730 [<ffffffff810d79b0>] ? mod_sysfs_teardown+0x150/0x150 [<ffffffff81183232>] ? __vmalloc+0x22/0x30 [<ffffffff810d73c0>] ? module_sect_show+0x30/0x30 [<ffffffff810dac84>] SyS_init_module+0x94/0xc0 [<ffffffff815d3997>] system_call_fastpath+0x12/0x6a Code: 66 66 66 90 b9 1e 00 00 00 48 89 f0 48 8d 77 08 48 89 c7 f3 48 a5 c9 c3 0f 1f 00 55 48 89 e5 66 66 66 66 90 48 8b 87 e8 13 00 00 <8b> 80 20 01 00 00 c9 c3 0f 1f 84 00 00 00 00 00 55 48 89 e5 66 RIP [<ffffffffa06b9060>] ipoib_get_iflink+0x10/0x20 [ib_ipoib] RSP <ffff88047579f9b8> CR2: 0000000000000120 ---[ end trace a8610f6e9640eb85 ]--- Signed-off-by: Honggang Li <honli@redhat.com> --- drivers/infiniband/ulp/ipoib/ipoib_main.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/infiniband/ulp/ipoib/ipoib_main.c b/drivers/infiniband/ulp/ipoib/ipoib_main.c index 657b89b..11ea6e2 100644 --- a/drivers/infiniband/ulp/ipoib/ipoib_main.c +++ b/drivers/infiniband/ulp/ipoib/ipoib_main.c @@ -846,7 +846,10 @@ static int ipoib_get_iflink(const struct net_device *dev) { struct ipoib_dev_priv *priv = netdev_priv(dev); - return priv->parent->ifindex; + if (priv && priv->parent) + return priv->parent->ifindex; + else + return 0; } static u32 ipoib_addr_hash(struct ipoib_neigh_hash *htbl, u8 *daddr) -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 53+ messages in thread
[parent not found: <1429024817-21561-2-git-send-email-honli-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>]
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink 2015-04-14 15:20 ` [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink Honggang Li @ 2015-04-14 15:34 ` Eric Dumazet 2015-04-14 16:30 ` Erez Shitrit 1 sibling, 0 replies; 53+ messages in thread From: Eric Dumazet @ 2015-04-14 15:34 UTC (permalink / raw) To: Honggang Li Cc: roland-DgEjT+Ai2ygdnm+yROfE0A, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, kaber-dcUjhNyLwpNeoWH0uzbU5w, davem-fT/PcQaiUtIeIZ0/mPfg9Q, alex.estrin-ral2JQCrhuEAvxtiuMwx3w, dledford-H+wXaHxf7aLQT0dZR+AlfA, edumazet-hpIqsD4AKlfQT0dZR+AlfA, erezsh-VPRAkNaXOzVWk0Htik3J/w, nicolas.dichtel-pdR9zngts4EAvxtiuMwx3w, maheshb-hpIqsD4AKlfQT0dZR+AlfA, jbenc-H+wXaHxf7aLQT0dZR+AlfA, ebiederm-aS9lmoZGLiVWk0Htik3J/w, elfring-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f, f.fainelli-Re5JQEeQqe8AvxtiuMwx3w, linux-0h96xk9xTtrk1uMJSBkQmQ, andrew-g2DYL2Zd6BY, sfeldma-Re5JQEeQqe8AvxtiuMwx3w, alexander.h.duyck-ral2JQCrhuEAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA On Tue, 2015-04-14 at 23:20 +0800, Honggang Li wrote: > Starting monitoring for VG vg_rdma01: 3 logical volume(s) in volume > group "vg_rdma01" monitored > [ OK ] > CR2: 0000000000000120 > ---[ end trace a8610f6e9640eb85 ]--- > > Signed-off-by: Honggang Li <honli-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> When was this bug added ? Please add a proper Fixes: tag -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink @ 2015-04-14 15:34 ` Eric Dumazet 0 siblings, 0 replies; 53+ messages in thread From: Eric Dumazet @ 2015-04-14 15:34 UTC (permalink / raw) To: Honggang Li Cc: roland, sean.hefty, hal.rosenstock, kaber, davem, alex.estrin, dledford, edumazet, erezsh, nicolas.dichtel, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev On Tue, 2015-04-14 at 23:20 +0800, Honggang Li wrote: > Starting monitoring for VG vg_rdma01: 3 logical volume(s) in volume > group "vg_rdma01" monitored > [ OK ] > CR2: 0000000000000120 > ---[ end trace a8610f6e9640eb85 ]--- > > Signed-off-by: Honggang Li <honli@redhat.com> When was this bug added ? Please add a proper Fixes: tag ^ permalink raw reply [flat|nested] 53+ messages in thread
[parent not found: <1429025673.7346.37.camel-XN9IlZ5yJG9HTL0Zs8A6p/gx64E7kk8eUsxypvmhUTTZJqsBc5GL+g@public.gmane.org>]
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink 2015-04-14 15:34 ` Eric Dumazet @ 2015-04-14 15:44 ` Honggang LI -1 siblings, 0 replies; 53+ messages in thread From: Honggang LI @ 2015-04-14 15:44 UTC (permalink / raw) To: Eric Dumazet Cc: roland-DgEjT+Ai2ygdnm+yROfE0A, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, kaber-dcUjhNyLwpNeoWH0uzbU5w, davem-fT/PcQaiUtIeIZ0/mPfg9Q, alex.estrin-ral2JQCrhuEAvxtiuMwx3w, dledford-H+wXaHxf7aLQT0dZR+AlfA, edumazet-hpIqsD4AKlfQT0dZR+AlfA, erezsh-VPRAkNaXOzVWk0Htik3J/w, nicolas.dichtel-pdR9zngts4EAvxtiuMwx3w, maheshb-hpIqsD4AKlfQT0dZR+AlfA, jbenc-H+wXaHxf7aLQT0dZR+AlfA, ebiederm-aS9lmoZGLiVWk0Htik3J/w, elfring-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f, f.fainelli-Re5JQEeQqe8AvxtiuMwx3w, linux-0h96xk9xTtrk1uMJSBkQmQ, andrew-g2DYL2Zd6BY, sfeldma-Re5JQEeQqe8AvxtiuMwx3w, alexander.h.duyck-ral2JQCrhuEAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA On Tue, Apr 14, 2015 at 08:34:33AM -0700, Eric Dumazet wrote: > On Tue, 2015-04-14 at 23:20 +0800, Honggang Li wrote: > > Starting monitoring for VG vg_rdma01: 3 logical volume(s) in volume > > group "vg_rdma01" monitored > > [ OK ] > > > > CR2: 0000000000000120 > > ---[ end trace a8610f6e9640eb85 ]--- > > > > Signed-off-by: Honggang Li <honli-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> > > When was this bug added ? > Sorry, I do not know. I ran into this bug today when I'm tracing a race condition issue related qib. In order to check upstream linux-next fix the race condition or not, I build linux-next-20150414 on two machines. Both machines panic at modprobe ib_ipoib. Do you means I need to report a bug? But I do not know report to who or where. thanks > Please add a proper Fixes: tag > > -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink @ 2015-04-14 15:44 ` Honggang LI 0 siblings, 0 replies; 53+ messages in thread From: Honggang LI @ 2015-04-14 15:44 UTC (permalink / raw) To: Eric Dumazet Cc: roland, sean.hefty, hal.rosenstock, kaber, davem, alex.estrin, dledford, edumazet, erezsh, nicolas.dichtel, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev On Tue, Apr 14, 2015 at 08:34:33AM -0700, Eric Dumazet wrote: > On Tue, 2015-04-14 at 23:20 +0800, Honggang Li wrote: > > Starting monitoring for VG vg_rdma01: 3 logical volume(s) in volume > > group "vg_rdma01" monitored > > [ OK ] > > > > CR2: 0000000000000120 > > ---[ end trace a8610f6e9640eb85 ]--- > > > > Signed-off-by: Honggang Li <honli@redhat.com> > > When was this bug added ? > Sorry, I do not know. I ran into this bug today when I'm tracing a race condition issue related qib. In order to check upstream linux-next fix the race condition or not, I build linux-next-20150414 on two machines. Both machines panic at modprobe ib_ipoib. Do you means I need to report a bug? But I do not know report to who or where. thanks > Please add a proper Fixes: tag > > ^ permalink raw reply [flat|nested] 53+ messages in thread
[parent not found: <20150414154422.GB21856-9l7K0WC0B0wP68cbUhXDDlaTQe2KTcn/@public.gmane.org>]
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink 2015-04-14 15:44 ` Honggang LI @ 2015-04-14 15:49 ` Nicolas Dichtel -1 siblings, 0 replies; 53+ messages in thread From: Nicolas Dichtel @ 2015-04-14 15:49 UTC (permalink / raw) To: Honggang LI, Eric Dumazet Cc: roland-DgEjT+Ai2ygdnm+yROfE0A, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, kaber-dcUjhNyLwpNeoWH0uzbU5w, davem-fT/PcQaiUtIeIZ0/mPfg9Q, alex.estrin-ral2JQCrhuEAvxtiuMwx3w, dledford-H+wXaHxf7aLQT0dZR+AlfA, edumazet-hpIqsD4AKlfQT0dZR+AlfA, erezsh-VPRAkNaXOzVWk0Htik3J/w, maheshb-hpIqsD4AKlfQT0dZR+AlfA, jbenc-H+wXaHxf7aLQT0dZR+AlfA, ebiederm-aS9lmoZGLiVWk0Htik3J/w, elfring-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f, f.fainelli-Re5JQEeQqe8AvxtiuMwx3w, linux-0h96xk9xTtrk1uMJSBkQmQ, andrew-g2DYL2Zd6BY, sfeldma-Re5JQEeQqe8AvxtiuMwx3w, alexander.h.duyck-ral2JQCrhuEAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA Le 14/04/2015 17:44, Honggang LI a écrit : > On Tue, Apr 14, 2015 at 08:34:33AM -0700, Eric Dumazet wrote: >> On Tue, 2015-04-14 at 23:20 +0800, Honggang Li wrote: >>> Starting monitoring for VG vg_rdma01: 3 logical volume(s) in volume >>> group "vg_rdma01" monitored >>> [ OK ] >> >> >>> CR2: 0000000000000120 >>> ---[ end trace a8610f6e9640eb85 ]--- >>> >>> Signed-off-by: Honggang Li <honli-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> >> >> When was this bug added ? >> > > Sorry, I do not know. I ran into this bug today when I'm tracing a race > condition issue related qib. In order to check upstream linux-next fix > the race condition or not, I build linux-next-20150414 on two machines. Both > machines panic at modprobe ib_ipoib. Do you means I need to report a > bug? But I do not know report to who or where. Here is the tag: Fixes: 5aa7add8f14b ("infiniband/ipoib: implement ndo_get_iflink") -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink @ 2015-04-14 15:49 ` Nicolas Dichtel 0 siblings, 0 replies; 53+ messages in thread From: Nicolas Dichtel @ 2015-04-14 15:49 UTC (permalink / raw) To: Honggang LI, Eric Dumazet Cc: roland, sean.hefty, hal.rosenstock, kaber, davem, alex.estrin, dledford, edumazet, erezsh, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev Le 14/04/2015 17:44, Honggang LI a écrit : > On Tue, Apr 14, 2015 at 08:34:33AM -0700, Eric Dumazet wrote: >> On Tue, 2015-04-14 at 23:20 +0800, Honggang Li wrote: >>> Starting monitoring for VG vg_rdma01: 3 logical volume(s) in volume >>> group "vg_rdma01" monitored >>> [ OK ] >> >> >>> CR2: 0000000000000120 >>> ---[ end trace a8610f6e9640eb85 ]--- >>> >>> Signed-off-by: Honggang Li <honli@redhat.com> >> >> When was this bug added ? >> > > Sorry, I do not know. I ran into this bug today when I'm tracing a race > condition issue related qib. In order to check upstream linux-next fix > the race condition or not, I build linux-next-20150414 on two machines. Both > machines panic at modprobe ib_ipoib. Do you means I need to report a > bug? But I do not know report to who or where. Here is the tag: Fixes: 5aa7add8f14b ("infiniband/ipoib: implement ndo_get_iflink") ^ permalink raw reply [flat|nested] 53+ messages in thread
[parent not found: <552D3723.9050706-pdR9zngts4EAvxtiuMwx3w@public.gmane.org>]
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink 2015-04-14 15:49 ` Nicolas Dichtel @ 2015-04-14 15:53 ` Honggang LI -1 siblings, 0 replies; 53+ messages in thread From: Honggang LI @ 2015-04-14 15:53 UTC (permalink / raw) To: Nicolas Dichtel Cc: Eric Dumazet, roland-DgEjT+Ai2ygdnm+yROfE0A, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, kaber-dcUjhNyLwpNeoWH0uzbU5w, davem-fT/PcQaiUtIeIZ0/mPfg9Q, alex.estrin-ral2JQCrhuEAvxtiuMwx3w, dledford-H+wXaHxf7aLQT0dZR+AlfA, edumazet-hpIqsD4AKlfQT0dZR+AlfA, erezsh-VPRAkNaXOzVWk0Htik3J/w, maheshb-hpIqsD4AKlfQT0dZR+AlfA, jbenc-H+wXaHxf7aLQT0dZR+AlfA, ebiederm-aS9lmoZGLiVWk0Htik3J/w, elfring-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f, f.fainelli-Re5JQEeQqe8AvxtiuMwx3w, linux-0h96xk9xTtrk1uMJSBkQmQ, andrew-g2DYL2Zd6BY, sfeldma-Re5JQEeQqe8AvxtiuMwx3w, alexander.h.duyck-ral2JQCrhuEAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA On Tue, Apr 14, 2015 at 05:49:55PM +0200, Nicolas Dichtel wrote: > Le 14/04/2015 17:44, Honggang LI a écrit : > >On Tue, Apr 14, 2015 at 08:34:33AM -0700, Eric Dumazet wrote: > >>On Tue, 2015-04-14 at 23:20 +0800, Honggang Li wrote: > >>>Starting monitoring for VG vg_rdma01: 3 logical volume(s) in volume > >>>group "vg_rdma01" monitored > >>>[ OK ] > >> > >> > >>>CR2: 0000000000000120 > >>>---[ end trace a8610f6e9640eb85 ]--- > >>> > >>>Signed-off-by: Honggang Li <honli-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> > >> > >>When was this bug added ? > >> > > > >Sorry, I do not know. I ran into this bug today when I'm tracing a race > >condition issue related qib. In order to check upstream linux-next fix > >the race condition or not, I build linux-next-20150414 on two machines. Both > >machines panic at modprobe ib_ipoib. Do you means I need to report a > >bug? But I do not know report to who or where. > > Here is the tag: > Fixes: 5aa7add8f14b ("infiniband/ipoib: implement ndo_get_iflink") > thank you. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink @ 2015-04-14 15:53 ` Honggang LI 0 siblings, 0 replies; 53+ messages in thread From: Honggang LI @ 2015-04-14 15:53 UTC (permalink / raw) To: Nicolas Dichtel Cc: Eric Dumazet, roland, sean.hefty, hal.rosenstock, kaber, davem, alex.estrin, dledford, edumazet, erezsh, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev On Tue, Apr 14, 2015 at 05:49:55PM +0200, Nicolas Dichtel wrote: > Le 14/04/2015 17:44, Honggang LI a écrit : > >On Tue, Apr 14, 2015 at 08:34:33AM -0700, Eric Dumazet wrote: > >>On Tue, 2015-04-14 at 23:20 +0800, Honggang Li wrote: > >>>Starting monitoring for VG vg_rdma01: 3 logical volume(s) in volume > >>>group "vg_rdma01" monitored > >>>[ OK ] > >> > >> > >>>CR2: 0000000000000120 > >>>---[ end trace a8610f6e9640eb85 ]--- > >>> > >>>Signed-off-by: Honggang Li <honli@redhat.com> > >> > >>When was this bug added ? > >> > > > >Sorry, I do not know. I ran into this bug today when I'm tracing a race > >condition issue related qib. In order to check upstream linux-next fix > >the race condition or not, I build linux-next-20150414 on two machines. Both > >machines panic at modprobe ib_ipoib. Do you means I need to report a > >bug? But I do not know report to who or where. > > Here is the tag: > Fixes: 5aa7add8f14b ("infiniband/ipoib: implement ndo_get_iflink") > thank you. ^ permalink raw reply [flat|nested] 53+ messages in thread
[parent not found: <20150414155307.GD21856-9l7K0WC0B0wP68cbUhXDDlaTQe2KTcn/@public.gmane.org>]
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink 2015-04-14 15:53 ` Honggang LI @ 2015-04-14 16:14 ` Eric Dumazet -1 siblings, 0 replies; 53+ messages in thread From: Eric Dumazet @ 2015-04-14 16:14 UTC (permalink / raw) To: Honggang LI Cc: Nicolas Dichtel, roland-DgEjT+Ai2ygdnm+yROfE0A, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, kaber-dcUjhNyLwpNeoWH0uzbU5w, davem-fT/PcQaiUtIeIZ0/mPfg9Q, alex.estrin-ral2JQCrhuEAvxtiuMwx3w, dledford-H+wXaHxf7aLQT0dZR+AlfA, edumazet-hpIqsD4AKlfQT0dZR+AlfA, erezsh-VPRAkNaXOzVWk0Htik3J/w, maheshb-hpIqsD4AKlfQT0dZR+AlfA, jbenc-H+wXaHxf7aLQT0dZR+AlfA, ebiederm-aS9lmoZGLiVWk0Htik3J/w, elfring-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f, f.fainelli-Re5JQEeQqe8AvxtiuMwx3w, linux-0h96xk9xTtrk1uMJSBkQmQ, andrew-g2DYL2Zd6BY, sfeldma-Re5JQEeQqe8AvxtiuMwx3w, alexander.h.duyck-ral2JQCrhuEAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA On Tue, 2015-04-14 at 23:53 +0800, Honggang LI wrote: > On Tue, Apr 14, 2015 at 05:49:55PM +0200, Nicolas Dichtel wrote: > > Le 14/04/2015 17:44, Honggang LI a écrit : > > >On Tue, Apr 14, 2015 at 08:34:33AM -0700, Eric Dumazet wrote: > > >>On Tue, 2015-04-14 at 23:20 +0800, Honggang Li wrote: > > >>>Starting monitoring for VG vg_rdma01: 3 logical volume(s) in volume > > >>>group "vg_rdma01" monitored > > >>>[ OK ] > > >> > > >> > > >>>CR2: 0000000000000120 > > >>>---[ end trace a8610f6e9640eb85 ]--- > > >>> > > >>>Signed-off-by: Honggang Li <honli-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> > > >> > > >>When was this bug added ? > > >> > > > > > >Sorry, I do not know. I ran into this bug today when I'm tracing a race > > >condition issue related qib. In order to check upstream linux-next fix > > >the race condition or not, I build linux-next-20150414 on two machines. Both > > >machines panic at modprobe ib_ipoib. Do you means I need to report a > > >bug? But I do not know report to who or where. > > > > Here is the tag: > > Fixes: 5aa7add8f14b ("infiniband/ipoib: implement ndo_get_iflink") > > > > thank you. By adding a proper tag you : 1) Assert you did a research in git history/blame to find original commit. 2) You looked at other similar bugs added by this commit 3) Add meta information in git history to ease backports 4) CC original author to let him/her know he made a mistake and increase his/her knowledge. 5) Help maintainers -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink @ 2015-04-14 16:14 ` Eric Dumazet 0 siblings, 0 replies; 53+ messages in thread From: Eric Dumazet @ 2015-04-14 16:14 UTC (permalink / raw) To: Honggang LI Cc: Nicolas Dichtel, roland, sean.hefty, hal.rosenstock, kaber, davem, alex.estrin, dledford, edumazet, erezsh, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev On Tue, 2015-04-14 at 23:53 +0800, Honggang LI wrote: > On Tue, Apr 14, 2015 at 05:49:55PM +0200, Nicolas Dichtel wrote: > > Le 14/04/2015 17:44, Honggang LI a écrit : > > >On Tue, Apr 14, 2015 at 08:34:33AM -0700, Eric Dumazet wrote: > > >>On Tue, 2015-04-14 at 23:20 +0800, Honggang Li wrote: > > >>>Starting monitoring for VG vg_rdma01: 3 logical volume(s) in volume > > >>>group "vg_rdma01" monitored > > >>>[ OK ] > > >> > > >> > > >>>CR2: 0000000000000120 > > >>>---[ end trace a8610f6e9640eb85 ]--- > > >>> > > >>>Signed-off-by: Honggang Li <honli@redhat.com> > > >> > > >>When was this bug added ? > > >> > > > > > >Sorry, I do not know. I ran into this bug today when I'm tracing a race > > >condition issue related qib. In order to check upstream linux-next fix > > >the race condition or not, I build linux-next-20150414 on two machines. Both > > >machines panic at modprobe ib_ipoib. Do you means I need to report a > > >bug? But I do not know report to who or where. > > > > Here is the tag: > > Fixes: 5aa7add8f14b ("infiniband/ipoib: implement ndo_get_iflink") > > > > thank you. By adding a proper tag you : 1) Assert you did a research in git history/blame to find original commit. 2) You looked at other similar bugs added by this commit 3) Add meta information in git history to ease backports 4) CC original author to let him/her know he made a mistake and increase his/her knowledge. 5) Help maintainers ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink 2015-04-14 15:49 ` Nicolas Dichtel @ 2015-04-14 16:01 ` Yann Droneaud -1 siblings, 0 replies; 53+ messages in thread From: Yann Droneaud @ 2015-04-14 16:01 UTC (permalink / raw) To: nicolas.dichtel-pdR9zngts4EAvxtiuMwx3w Cc: Honggang LI, Eric Dumazet, roland-DgEjT+Ai2ygdnm+yROfE0A, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, kaber-dcUjhNyLwpNeoWH0uzbU5w, davem-fT/PcQaiUtIeIZ0/mPfg9Q, alex.estrin-ral2JQCrhuEAvxtiuMwx3w, dledford-H+wXaHxf7aLQT0dZR+AlfA, edumazet-hpIqsD4AKlfQT0dZR+AlfA, erezsh-VPRAkNaXOzVWk0Htik3J/w, maheshb-hpIqsD4AKlfQT0dZR+AlfA, jbenc-H+wXaHxf7aLQT0dZR+AlfA, ebiederm-aS9lmoZGLiVWk0Htik3J/w, elfring-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f, f.fainelli-Re5JQEeQqe8AvxtiuMwx3w, linux-0h96xk9xTtrk1uMJSBkQmQ, andrew-g2DYL2Zd6BY, sfeldma-Re5JQEeQqe8AvxtiuMwx3w, alexander.h.duyck-ral2JQCrhuEAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA Hi Nicolas, Le mardi 14 avril 2015 à 17:49 +0200, Nicolas Dichtel a écrit : > Le 14/04/2015 17:44, Honggang LI a écrit : > > On Tue, Apr 14, 2015 at 08:34:33AM -0700, Eric Dumazet wrote: > >> On Tue, 2015-04-14 at 23:20 +0800, Honggang Li wrote: > >>> Starting monitoring for VG vg_rdma01: 3 logical volume(s) in volume > >>> group "vg_rdma01" monitored > >>> [ OK ] > >> > >> > >>> CR2: 0000000000000120 > >>> ---[ end trace a8610f6e9640eb85 ]--- > >>> > >>> Signed-off-by: Honggang Li <honli-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> > >> > >> When was this bug added ? > >> > > > > Sorry, I do not know. I ran into this bug today when I'm tracing a race > > condition issue related qib. In order to check upstream linux-next fix > > the race condition or not, I build linux-next-20150414 on two machines. Both > > machines panic at modprobe ib_ipoib. Do you means I need to report a > > bug? But I do not know report to who or where. > > Here is the tag: > Fixes: 5aa7add8f14b ("infiniband/ipoib: implement ndo_get_iflink") > Pardon me, but this patch was never submitted to linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org for review !? Regards. -- Yann Droneaud OPTEYA -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink @ 2015-04-14 16:01 ` Yann Droneaud 0 siblings, 0 replies; 53+ messages in thread From: Yann Droneaud @ 2015-04-14 16:01 UTC (permalink / raw) To: nicolas.dichtel Cc: Honggang LI, Eric Dumazet, roland, sean.hefty, hal.rosenstock, kaber, davem, alex.estrin, dledford, edumazet, erezsh, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev Hi Nicolas, Le mardi 14 avril 2015 à 17:49 +0200, Nicolas Dichtel a écrit : > Le 14/04/2015 17:44, Honggang LI a écrit : > > On Tue, Apr 14, 2015 at 08:34:33AM -0700, Eric Dumazet wrote: > >> On Tue, 2015-04-14 at 23:20 +0800, Honggang Li wrote: > >>> Starting monitoring for VG vg_rdma01: 3 logical volume(s) in volume > >>> group "vg_rdma01" monitored > >>> [ OK ] > >> > >> > >>> CR2: 0000000000000120 > >>> ---[ end trace a8610f6e9640eb85 ]--- > >>> > >>> Signed-off-by: Honggang Li <honli@redhat.com> > >> > >> When was this bug added ? > >> > > > > Sorry, I do not know. I ran into this bug today when I'm tracing a race > > condition issue related qib. In order to check upstream linux-next fix > > the race condition or not, I build linux-next-20150414 on two machines. Both > > machines panic at modprobe ib_ipoib. Do you means I need to report a > > bug? But I do not know report to who or where. > > Here is the tag: > Fixes: 5aa7add8f14b ("infiniband/ipoib: implement ndo_get_iflink") > Pardon me, but this patch was never submitted to linux-rdma@vger.kernel.org for review !? Regards. -- Yann Droneaud OPTEYA ^ permalink raw reply [flat|nested] 53+ messages in thread
[parent not found: <1429027293.4333.5.camel-RlY5vtjFyJ3QT0dZR+AlfA@public.gmane.org>]
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink 2015-04-14 16:01 ` Yann Droneaud @ 2015-04-14 16:44 ` Nicolas Dichtel -1 siblings, 0 replies; 53+ messages in thread From: Nicolas Dichtel @ 2015-04-14 16:44 UTC (permalink / raw) To: Yann Droneaud Cc: Honggang LI, Eric Dumazet, roland-DgEjT+Ai2ygdnm+yROfE0A, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, kaber-dcUjhNyLwpNeoWH0uzbU5w, davem-fT/PcQaiUtIeIZ0/mPfg9Q, alex.estrin-ral2JQCrhuEAvxtiuMwx3w, dledford-H+wXaHxf7aLQT0dZR+AlfA, edumazet-hpIqsD4AKlfQT0dZR+AlfA, erezsh-VPRAkNaXOzVWk0Htik3J/w, maheshb-hpIqsD4AKlfQT0dZR+AlfA, jbenc-H+wXaHxf7aLQT0dZR+AlfA, ebiederm-aS9lmoZGLiVWk0Htik3J/w, elfring-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f, f.fainelli-Re5JQEeQqe8AvxtiuMwx3w, linux-0h96xk9xTtrk1uMJSBkQmQ, andrew-g2DYL2Zd6BY, sfeldma-Re5JQEeQqe8AvxtiuMwx3w, alexander.h.duyck-ral2JQCrhuEAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA Le 14/04/2015 18:01, Yann Droneaud a écrit : [snip] >> Here is the tag: >> Fixes: 5aa7add8f14b ("infiniband/ipoib: implement ndo_get_iflink") >> > > Pardon me, but this patch was never submitted to > linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org for review !? Sorry for that, I missed it. Only Roland Dreier was CCed. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink @ 2015-04-14 16:44 ` Nicolas Dichtel 0 siblings, 0 replies; 53+ messages in thread From: Nicolas Dichtel @ 2015-04-14 16:44 UTC (permalink / raw) To: Yann Droneaud Cc: Honggang LI, Eric Dumazet, roland, sean.hefty, hal.rosenstock, kaber, davem, alex.estrin, dledford, edumazet, erezsh, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev Le 14/04/2015 18:01, Yann Droneaud a écrit : [snip] >> Here is the tag: >> Fixes: 5aa7add8f14b ("infiniband/ipoib: implement ndo_get_iflink") >> > > Pardon me, but this patch was never submitted to > linux-rdma@vger.kernel.org for review !? Sorry for that, I missed it. Only Roland Dreier was CCed. ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink 2015-04-14 15:20 ` [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink Honggang Li [not found] ` <1429024817-21561-2-git-send-email-honli-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> @ 2015-04-14 16:30 ` Erez Shitrit [not found] ` <CAAk-MO-O9sjHQvDfCEuzJJPvUMXJTuRaCzrCkB0xc1DUfK8Aew-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 1 sibling, 1 reply; 53+ messages in thread From: Erez Shitrit @ 2015-04-14 16:30 UTC (permalink / raw) To: Honggang Li Cc: Roland Dreier, sean.hefty, hal.rosenstock, kaber, davem, Alex Estrin, Doug Ledford, edumazet, Erez Shitrit, nicolas.dichtel, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev On Tue, Apr 14, 2015 at 6:20 PM, Honggang Li <honli@redhat.com> wrote: > [...] Hi, > diff --git a/drivers/infiniband/ulp/ipoib/ipoib_main.c b/drivers/infiniband/ulp/ipoib/ipoib_main.c > index 657b89b..11ea6e2 100644 > --- a/drivers/infiniband/ulp/ipoib/ipoib_main.c > +++ b/drivers/infiniband/ulp/ipoib/ipoib_main.c > @@ -846,7 +846,10 @@ static int ipoib_get_iflink(const struct net_device *dev) > { > struct ipoib_dev_priv *priv = netdev_priv(dev); > > - return priv->parent->ifindex; > + if (priv && priv->parent) > + return priv->parent->ifindex; > + else > + return 0; This will make parent interface to return 0 instead of its own ifindex. I would suggest write something like that: + /* parent interface */ + if (!test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags)) + return dev->ifindex; + + /* child/vlan interface */ + if (!priv->parent) + return -1; + return priv->parent->ifindex; Thanks, Erez. > } > > static u32 ipoib_addr_hash(struct ipoib_neigh_hash *htbl, u8 *daddr) > -- > 1.8.3.1 > ^ permalink raw reply [flat|nested] 53+ messages in thread
[parent not found: <CAAk-MO-O9sjHQvDfCEuzJJPvUMXJTuRaCzrCkB0xc1DUfK8Aew-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink 2015-04-14 16:30 ` Erez Shitrit @ 2015-04-14 16:46 ` Nicolas Dichtel 0 siblings, 0 replies; 53+ messages in thread From: Nicolas Dichtel @ 2015-04-14 16:46 UTC (permalink / raw) To: Erez Shitrit, Honggang Li Cc: Roland Dreier, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, kaber-dcUjhNyLwpNeoWH0uzbU5w, davem-fT/PcQaiUtIeIZ0/mPfg9Q, Alex Estrin, Doug Ledford, edumazet-hpIqsD4AKlfQT0dZR+AlfA, Erez Shitrit, maheshb-hpIqsD4AKlfQT0dZR+AlfA, jbenc-H+wXaHxf7aLQT0dZR+AlfA, ebiederm-aS9lmoZGLiVWk0Htik3J/w, elfring-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f, f.fainelli-Re5JQEeQqe8AvxtiuMwx3w, linux-0h96xk9xTtrk1uMJSBkQmQ, andrew-g2DYL2Zd6BY, sfeldma-Re5JQEeQqe8AvxtiuMwx3w, alexander.h.duyck-ral2JQCrhuEAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA Le 14/04/2015 18:30, Erez Shitrit a écrit : > On Tue, Apr 14, 2015 at 6:20 PM, Honggang Li <honli-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> wrote: [snip] > This will make parent interface to return 0 instead of its own ifindex. > I would suggest write something like that: > > + /* parent interface */ > + if (!test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags)) > + return dev->ifindex; > + > + /* child/vlan interface */ > + if (!priv->parent) > + return -1; 'return 0' here. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink @ 2015-04-14 16:46 ` Nicolas Dichtel 0 siblings, 0 replies; 53+ messages in thread From: Nicolas Dichtel @ 2015-04-14 16:46 UTC (permalink / raw) To: Erez Shitrit, Honggang Li Cc: Roland Dreier, sean.hefty, hal.rosenstock, kaber, davem, Alex Estrin, Doug Ledford, edumazet, Erez Shitrit, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev Le 14/04/2015 18:30, Erez Shitrit a écrit : > On Tue, Apr 14, 2015 at 6:20 PM, Honggang Li <honli@redhat.com> wrote: [snip] > This will make parent interface to return 0 instead of its own ifindex. > I would suggest write something like that: > > + /* parent interface */ > + if (!test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags)) > + return dev->ifindex; > + > + /* child/vlan interface */ > + if (!priv->parent) > + return -1; 'return 0' here. ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink 2015-04-14 16:30 ` Erez Shitrit @ 2015-04-14 20:41 ` Jason Gunthorpe 0 siblings, 0 replies; 53+ messages in thread From: Jason Gunthorpe @ 2015-04-14 20:41 UTC (permalink / raw) To: Erez Shitrit Cc: Honggang Li, Roland Dreier, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, kaber-dcUjhNyLwpNeoWH0uzbU5w, davem-fT/PcQaiUtIeIZ0/mPfg9Q, Alex Estrin, Doug Ledford, edumazet-hpIqsD4AKlfQT0dZR+AlfA, Erez Shitrit, nicolas.dichtel-pdR9zngts4EAvxtiuMwx3w, maheshb-hpIqsD4AKlfQT0dZR+AlfA, jbenc-H+wXaHxf7aLQT0dZR+AlfA, ebiederm-aS9lmoZGLiVWk0Htik3J/w, elfring-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f, f.fainelli-Re5JQEeQqe8AvxtiuMwx3w, linux-0h96xk9xTtrk1uMJSBkQmQ, andrew-g2DYL2Zd6BY, sfeldma-Re5JQEeQqe8AvxtiuMwx3w, alexander.h.duyck-ral2JQCrhuEAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA On Tue, Apr 14, 2015 at 07:30:03PM +0300, Erez Shitrit wrote: > > diff --git a/drivers/infiniband/ulp/ipoib/ipoib_main.c b/drivers/infiniband/ulp/ipoib/ipoib_main.c > > index 657b89b..11ea6e2 100644 > > +++ b/drivers/infiniband/ulp/ipoib/ipoib_main.c > > @@ -846,7 +846,10 @@ static int ipoib_get_iflink(const struct net_device *dev) > > { > > struct ipoib_dev_priv *priv = netdev_priv(dev); > > > > - return priv->parent->ifindex; > > + if (priv && priv->parent) > > + return priv->parent->ifindex; > > + else > > + return 0; > This will make parent interface to return 0 instead of its own ifindex. > I would suggest write something like that: Agree > + /* parent interface */ > + if (!test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags)) > + return dev->ifindex; > + > + /* child/vlan interface */ > + if (!priv->parent) > + return -1; Like was said for other drivers, I can't see how parent can be null while IPOIB_FLAG_SUBINTERFACE is set. Drop the last if. Erez, you basically rewrote this, please make a proper patch with the Fixes and Reported-By credit for Honggang. Lets merge this through Dave M's tree right away. Thank you all Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink @ 2015-04-14 20:41 ` Jason Gunthorpe 0 siblings, 0 replies; 53+ messages in thread From: Jason Gunthorpe @ 2015-04-14 20:41 UTC (permalink / raw) To: Erez Shitrit Cc: Honggang Li, Roland Dreier, sean.hefty, hal.rosenstock, kaber, davem, Alex Estrin, Doug Ledford, edumazet, Erez Shitrit, nicolas.dichtel, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev On Tue, Apr 14, 2015 at 07:30:03PM +0300, Erez Shitrit wrote: > > diff --git a/drivers/infiniband/ulp/ipoib/ipoib_main.c b/drivers/infiniband/ulp/ipoib/ipoib_main.c > > index 657b89b..11ea6e2 100644 > > +++ b/drivers/infiniband/ulp/ipoib/ipoib_main.c > > @@ -846,7 +846,10 @@ static int ipoib_get_iflink(const struct net_device *dev) > > { > > struct ipoib_dev_priv *priv = netdev_priv(dev); > > > > - return priv->parent->ifindex; > > + if (priv && priv->parent) > > + return priv->parent->ifindex; > > + else > > + return 0; > This will make parent interface to return 0 instead of its own ifindex. > I would suggest write something like that: Agree > + /* parent interface */ > + if (!test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags)) > + return dev->ifindex; > + > + /* child/vlan interface */ > + if (!priv->parent) > + return -1; Like was said for other drivers, I can't see how parent can be null while IPOIB_FLAG_SUBINTERFACE is set. Drop the last if. Erez, you basically rewrote this, please make a proper patch with the Fixes and Reported-By credit for Honggang. Lets merge this through Dave M's tree right away. Thank you all Jason ^ permalink raw reply [flat|nested] 53+ messages in thread
[parent not found: <20150414204133.GJ7682-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>]
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink 2015-04-14 20:41 ` Jason Gunthorpe (?) @ 2015-04-15 5:24 ` Or Gerlitz -1 siblings, 0 replies; 53+ messages in thread From: Or Gerlitz @ 2015-04-15 5:24 UTC (permalink / raw) To: Erez Shitrit Cc: Jason Gunthorpe, Honggang Li, Roland Dreier, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, kaber-dcUjhNyLwpNeoWH0uzbU5w, davem-fT/PcQaiUtIeIZ0/mPfg9Q, Alex Estrin, Doug Ledford, edumazet-hpIqsD4AKlfQT0dZR+AlfA, Erez Shitrit, nicolas.dichtel-pdR9zngts4EAvxtiuMwx3w, maheshb-hpIqsD4AKlfQT0dZR+AlfA, jbenc-H+wXaHxf7aLQT0dZR+AlfA, ebiederm-aS9lmoZGLiVWk0Htik3J/w, elfring-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f, f.fainelli-Re5JQEeQqe8AvxtiuMwx3w, linux-0h96xk9xTtrk1uMJSBkQmQ, andrew-g2DYL2Zd6BY, sfeldma-Re5JQEeQqe8AvxtiuMwx3w, alexander.h.duyck-ral2JQCrhuEAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA On 4/14/2015 11:41 PM, Jason Gunthorpe wrote: > Erez, you basically rewrote this, please make a proper patch with the Fixes and Reported-By credit for Honggang. Lets merge this through Dave M's tree right away. Agree, Erez, add proper Fixes: XXX note and send a patch to netdev against net-next. No need for the lengthy crash dump there. Nicolas, next time you patch IPoIB, please cc linux-rdma. Or. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink @ 2015-04-15 5:24 ` Or Gerlitz 0 siblings, 0 replies; 53+ messages in thread From: Or Gerlitz @ 2015-04-15 5:24 UTC (permalink / raw) To: Erez Shitrit Cc: Jason Gunthorpe, Honggang Li, Roland Dreier, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, kaber-dcUjhNyLwpNeoWH0uzbU5w, davem-fT/PcQaiUtIeIZ0/mPfg9Q, Alex Estrin, Doug Ledford, edumazet-hpIqsD4AKlfQT0dZR+AlfA, Erez Shitrit, nicolas.dichtel-pdR9zngts4EAvxtiuMwx3w, maheshb-hpIqsD4AKlfQT0dZR+AlfA, jbenc-H+wXaHxf7aLQT0dZR+AlfA, ebiederm-aS9lmoZGLiVWk0Htik3J/w, elfring-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f, f.fainelli-Re5JQEeQqe8AvxtiuMwx3w, linux-0h96xk9xTtrk1uMJSBkQmQ, andrew-g2DYL2Zd6BY, sfeldma-Re5JQEeQqe8AvxtiuMwx3w, alexander.h.duyck-ral2JQCrhuEAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA On 4/14/2015 11:41 PM, Jason Gunthorpe wrote: > Erez, you basically rewrote this, please make a proper patch with the Fixes and Reported-By credit for Honggang. Lets merge this through Dave M's tree right away. Agree, Erez, add proper Fixes: XXX note and send a patch to netdev against net-next. No need for the lengthy crash dump there. Nicolas, next time you patch IPoIB, please cc linux-rdma. Or. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink @ 2015-04-15 5:24 ` Or Gerlitz 0 siblings, 0 replies; 53+ messages in thread From: Or Gerlitz @ 2015-04-15 5:24 UTC (permalink / raw) To: Erez Shitrit Cc: Jason Gunthorpe, Honggang Li, Roland Dreier, sean.hefty, hal.rosenstock, kaber, davem, Alex Estrin, Doug Ledford, edumazet, Erez Shitrit, nicolas.dichtel, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev On 4/14/2015 11:41 PM, Jason Gunthorpe wrote: > Erez, you basically rewrote this, please make a proper patch with the Fixes and Reported-By credit for Honggang. Lets merge this through Dave M's tree right away. Agree, Erez, add proper Fixes: XXX note and send a patch to netdev against net-next. No need for the lengthy crash dump there. Nicolas, next time you patch IPoIB, please cc linux-rdma. Or. ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink 2015-04-14 20:41 ` Jason Gunthorpe @ 2015-04-15 6:17 ` Erez Shitrit -1 siblings, 0 replies; 53+ messages in thread From: Erez Shitrit @ 2015-04-15 6:17 UTC (permalink / raw) To: Jason Gunthorpe Cc: Honggang Li, Roland Dreier, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, kaber-dcUjhNyLwpNeoWH0uzbU5w, davem-fT/PcQaiUtIeIZ0/mPfg9Q, Alex Estrin, Doug Ledford, edumazet-hpIqsD4AKlfQT0dZR+AlfA, Erez Shitrit, nicolas.dichtel-pdR9zngts4EAvxtiuMwx3w, maheshb-hpIqsD4AKlfQT0dZR+AlfA, jbenc-H+wXaHxf7aLQT0dZR+AlfA, ebiederm-aS9lmoZGLiVWk0Htik3J/w, elfring-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f, f.fainelli-Re5JQEeQqe8AvxtiuMwx3w, linux-0h96xk9xTtrk1uMJSBkQmQ, andrew-g2DYL2Zd6BY, sfeldma-Re5JQEeQqe8AvxtiuMwx3w, alexander.h.duyck-ral2JQCrhuEAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA On 4/14/2015 11:41 PM, Jason Gunthorpe wrote: > On Tue, Apr 14, 2015 at 07:30:03PM +0300, Erez Shitrit wrote: > >>> diff --git a/drivers/infiniband/ulp/ipoib/ipoib_main.c b/drivers/infiniband/ulp/ipoib/ipoib_main.c >>> index 657b89b..11ea6e2 100644 >>> +++ b/drivers/infiniband/ulp/ipoib/ipoib_main.c >>> @@ -846,7 +846,10 @@ static int ipoib_get_iflink(const struct net_device *dev) >>> { >>> struct ipoib_dev_priv *priv = netdev_priv(dev); >>> >>> - return priv->parent->ifindex; >>> + if (priv && priv->parent) >>> + return priv->parent->ifindex; >>> + else >>> + return 0; >> This will make parent interface to return 0 instead of its own ifindex. >> I would suggest write something like that: > Agree > >> + /* parent interface */ >> + if (!test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags)) >> + return dev->ifindex; >> + >> + /* child/vlan interface */ >> + if (!priv->parent) >> + return -1; > Like was said for other drivers, I can't see how parent can be null > while IPOIB_FLAG_SUBINTERFACE is set. Drop the last if. It can, at least for ipoib child interface (AKA "vlan"), you can't control the call for that ndo and it can be called before the parent was set. > Erez, you basically rewrote this, please make a proper patch with the > Fixes and Reported-By credit for Honggang. Lets merge this through > Dave M's tree right away. > > Thank you all > > Jason > . > -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink @ 2015-04-15 6:17 ` Erez Shitrit 0 siblings, 0 replies; 53+ messages in thread From: Erez Shitrit @ 2015-04-15 6:17 UTC (permalink / raw) To: Jason Gunthorpe Cc: Honggang Li, Roland Dreier, sean.hefty, hal.rosenstock, kaber, davem, Alex Estrin, Doug Ledford, edumazet, Erez Shitrit, nicolas.dichtel, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev On 4/14/2015 11:41 PM, Jason Gunthorpe wrote: > On Tue, Apr 14, 2015 at 07:30:03PM +0300, Erez Shitrit wrote: > >>> diff --git a/drivers/infiniband/ulp/ipoib/ipoib_main.c b/drivers/infiniband/ulp/ipoib/ipoib_main.c >>> index 657b89b..11ea6e2 100644 >>> +++ b/drivers/infiniband/ulp/ipoib/ipoib_main.c >>> @@ -846,7 +846,10 @@ static int ipoib_get_iflink(const struct net_device *dev) >>> { >>> struct ipoib_dev_priv *priv = netdev_priv(dev); >>> >>> - return priv->parent->ifindex; >>> + if (priv && priv->parent) >>> + return priv->parent->ifindex; >>> + else >>> + return 0; >> This will make parent interface to return 0 instead of its own ifindex. >> I would suggest write something like that: > Agree > >> + /* parent interface */ >> + if (!test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags)) >> + return dev->ifindex; >> + >> + /* child/vlan interface */ >> + if (!priv->parent) >> + return -1; > Like was said for other drivers, I can't see how parent can be null > while IPOIB_FLAG_SUBINTERFACE is set. Drop the last if. It can, at least for ipoib child interface (AKA "vlan"), you can't control the call for that ndo and it can be called before the parent was set. > Erez, you basically rewrote this, please make a proper patch with the > Fixes and Reported-By credit for Honggang. Lets merge this through > Dave M's tree right away. > > Thank you all > > Jason > . > ^ permalink raw reply [flat|nested] 53+ messages in thread
[parent not found: <552E026A.4020200-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org>]
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink 2015-04-15 6:17 ` Erez Shitrit @ 2015-04-15 16:06 ` Jason Gunthorpe -1 siblings, 0 replies; 53+ messages in thread From: Jason Gunthorpe @ 2015-04-15 16:06 UTC (permalink / raw) To: Erez Shitrit Cc: Honggang Li, Roland Dreier, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, kaber-dcUjhNyLwpNeoWH0uzbU5w, davem-fT/PcQaiUtIeIZ0/mPfg9Q, Alex Estrin, Doug Ledford, edumazet-hpIqsD4AKlfQT0dZR+AlfA, Erez Shitrit, nicolas.dichtel-pdR9zngts4EAvxtiuMwx3w, maheshb-hpIqsD4AKlfQT0dZR+AlfA, jbenc-H+wXaHxf7aLQT0dZR+AlfA, ebiederm-aS9lmoZGLiVWk0Htik3J/w, elfring-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f, f.fainelli-Re5JQEeQqe8AvxtiuMwx3w, linux-0h96xk9xTtrk1uMJSBkQmQ, andrew-g2DYL2Zd6BY, sfeldma-Re5JQEeQqe8AvxtiuMwx3w, alexander.h.duyck-ral2JQCrhuEAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA On Wed, Apr 15, 2015 at 09:17:14AM +0300, Erez Shitrit wrote: > >>+ /* parent interface */ > >>+ if (!test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags)) > >>+ return dev->ifindex; > >>+ > >>+ /* child/vlan interface */ > >>+ if (!priv->parent) > >>+ return -1; > >Like was said for other drivers, I can't see how parent can be null > >while IPOIB_FLAG_SUBINTERFACE is set. Drop the last if. > It can, at least for ipoib child interface (AKA "vlan"), you can't > control the call for that ndo and it can be called before the parent > was set. If the ndo can be called before the netdev private structures are fully prepared then we have another bug, and returning -1 or 0 is not the right answer anyhow. For safety, fold this into your patch. diff --git a/drivers/infiniband/ulp/ipoib/ipoib_vlan.c b/drivers/infiniband/ulp/ipoib/ipoib_vlan.c index 9fad7b5ac8b9..e62b007adf5d 100644 --- a/drivers/infiniband/ulp/ipoib/ipoib_vlan.c +++ b/drivers/infiniband/ulp/ipoib/ipoib_vlan.c @@ -58,6 +58,7 @@ int __ipoib_vlan_add(struct ipoib_dev_priv *ppriv, struct ipoib_dev_priv *priv, /* MTU will be reset when mcast join happens */ priv->dev->mtu = IPOIB_UD_MTU(priv->max_ib_mtu); priv->mcast_mtu = priv->admin_mtu = priv->dev->mtu; + priv->parent = ppriv->dev; set_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags); result = ipoib_set_dev_features(priv, ppriv->ca); @@ -84,8 +85,6 @@ int __ipoib_vlan_add(struct ipoib_dev_priv *ppriv, struct ipoib_dev_priv *priv, goto register_failed; } - priv->parent = ppriv->dev; - ipoib_create_debug_files(priv->dev); /* RTNL childs don't need proprietary sysfs entries */ -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink @ 2015-04-15 16:06 ` Jason Gunthorpe 0 siblings, 0 replies; 53+ messages in thread From: Jason Gunthorpe @ 2015-04-15 16:06 UTC (permalink / raw) To: Erez Shitrit Cc: Honggang Li, Roland Dreier, sean.hefty, hal.rosenstock, kaber, davem, Alex Estrin, Doug Ledford, edumazet, Erez Shitrit, nicolas.dichtel, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev On Wed, Apr 15, 2015 at 09:17:14AM +0300, Erez Shitrit wrote: > >>+ /* parent interface */ > >>+ if (!test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags)) > >>+ return dev->ifindex; > >>+ > >>+ /* child/vlan interface */ > >>+ if (!priv->parent) > >>+ return -1; > >Like was said for other drivers, I can't see how parent can be null > >while IPOIB_FLAG_SUBINTERFACE is set. Drop the last if. > It can, at least for ipoib child interface (AKA "vlan"), you can't > control the call for that ndo and it can be called before the parent > was set. If the ndo can be called before the netdev private structures are fully prepared then we have another bug, and returning -1 or 0 is not the right answer anyhow. For safety, fold this into your patch. diff --git a/drivers/infiniband/ulp/ipoib/ipoib_vlan.c b/drivers/infiniband/ulp/ipoib/ipoib_vlan.c index 9fad7b5ac8b9..e62b007adf5d 100644 --- a/drivers/infiniband/ulp/ipoib/ipoib_vlan.c +++ b/drivers/infiniband/ulp/ipoib/ipoib_vlan.c @@ -58,6 +58,7 @@ int __ipoib_vlan_add(struct ipoib_dev_priv *ppriv, struct ipoib_dev_priv *priv, /* MTU will be reset when mcast join happens */ priv->dev->mtu = IPOIB_UD_MTU(priv->max_ib_mtu); priv->mcast_mtu = priv->admin_mtu = priv->dev->mtu; + priv->parent = ppriv->dev; set_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags); result = ipoib_set_dev_features(priv, ppriv->ca); @@ -84,8 +85,6 @@ int __ipoib_vlan_add(struct ipoib_dev_priv *ppriv, struct ipoib_dev_priv *priv, goto register_failed; } - priv->parent = ppriv->dev; - ipoib_create_debug_files(priv->dev); /* RTNL childs don't need proprietary sysfs entries */ ^ permalink raw reply related [flat|nested] 53+ messages in thread
[parent not found: <20150415160623.GA4653-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>]
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink 2015-04-15 16:06 ` Jason Gunthorpe @ 2015-04-16 11:27 ` Erez Shitrit -1 siblings, 0 replies; 53+ messages in thread From: Erez Shitrit @ 2015-04-16 11:27 UTC (permalink / raw) To: Jason Gunthorpe Cc: Honggang Li, Roland Dreier, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, Patrick McHardy, davem-fT/PcQaiUtIeIZ0/mPfg9Q, Alex Estrin, Doug Ledford, Eric Dumazet, Erez Shitrit, Nicolas Dichtel, Mahesh Bandewar, jbenc-H+wXaHxf7aLQT0dZR+AlfA, ebiederm-aS9lmoZGLiVWk0Htik3J/w, elfring-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f, Florian Fainelli, linux-0h96xk9xTtrk1uMJSBkQmQ, andrew-g2DYL2Zd6BY, Scott Feldman, alexander.h.duyck-ral2JQCrhuEAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA On Wed, Apr 15, 2015 at 7:06 PM, Jason Gunthorpe <jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org> wrote: > On Wed, Apr 15, 2015 at 09:17:14AM +0300, Erez Shitrit wrote: >> >>+ /* parent interface */ >> >>+ if (!test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags)) >> >>+ return dev->ifindex; >> >>+ >> >>+ /* child/vlan interface */ >> >>+ if (!priv->parent) >> >>+ return -1; > >> >Like was said for other drivers, I can't see how parent can be null >> >while IPOIB_FLAG_SUBINTERFACE is set. Drop the last if. > >> It can, at least for ipoib child interface (AKA "vlan"), you can't >> control the call for that ndo and it can be called before the parent >> was set. > > If the ndo can be called before the netdev private structures are fully > prepared then we have another bug, and returning -1 or 0 is not the right > answer anyhow. > > For safety, fold this into your patch. OK, will do that. > > diff --git a/drivers/infiniband/ulp/ipoib/ipoib_vlan.c b/drivers/infiniband/ulp/ipoib/ipoib_vlan.c > index 9fad7b5ac8b9..e62b007adf5d 100644 > --- a/drivers/infiniband/ulp/ipoib/ipoib_vlan.c > +++ b/drivers/infiniband/ulp/ipoib/ipoib_vlan.c > @@ -58,6 +58,7 @@ int __ipoib_vlan_add(struct ipoib_dev_priv *ppriv, struct ipoib_dev_priv *priv, > /* MTU will be reset when mcast join happens */ > priv->dev->mtu = IPOIB_UD_MTU(priv->max_ib_mtu); > priv->mcast_mtu = priv->admin_mtu = priv->dev->mtu; > + priv->parent = ppriv->dev; > set_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags); > > result = ipoib_set_dev_features(priv, ppriv->ca); > @@ -84,8 +85,6 @@ int __ipoib_vlan_add(struct ipoib_dev_priv *ppriv, struct ipoib_dev_priv *priv, > goto register_failed; > } > > - priv->parent = ppriv->dev; > - > ipoib_create_debug_files(priv->dev); > > /* RTNL childs don't need proprietary sysfs entries */ -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink @ 2015-04-16 11:27 ` Erez Shitrit 0 siblings, 0 replies; 53+ messages in thread From: Erez Shitrit @ 2015-04-16 11:27 UTC (permalink / raw) To: Jason Gunthorpe Cc: Honggang Li, Roland Dreier, sean.hefty, hal.rosenstock, Patrick McHardy, davem, Alex Estrin, Doug Ledford, Eric Dumazet, Erez Shitrit, Nicolas Dichtel, Mahesh Bandewar, jbenc, ebiederm, elfring, Florian Fainelli, linux, andrew, Scott Feldman, alexander.h.duyck, linux-rdma, linux-kernel, netdev On Wed, Apr 15, 2015 at 7:06 PM, Jason Gunthorpe <jgunthorpe@obsidianresearch.com> wrote: > On Wed, Apr 15, 2015 at 09:17:14AM +0300, Erez Shitrit wrote: >> >>+ /* parent interface */ >> >>+ if (!test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags)) >> >>+ return dev->ifindex; >> >>+ >> >>+ /* child/vlan interface */ >> >>+ if (!priv->parent) >> >>+ return -1; > >> >Like was said for other drivers, I can't see how parent can be null >> >while IPOIB_FLAG_SUBINTERFACE is set. Drop the last if. > >> It can, at least for ipoib child interface (AKA "vlan"), you can't >> control the call for that ndo and it can be called before the parent >> was set. > > If the ndo can be called before the netdev private structures are fully > prepared then we have another bug, and returning -1 or 0 is not the right > answer anyhow. > > For safety, fold this into your patch. OK, will do that. > > diff --git a/drivers/infiniband/ulp/ipoib/ipoib_vlan.c b/drivers/infiniband/ulp/ipoib/ipoib_vlan.c > index 9fad7b5ac8b9..e62b007adf5d 100644 > --- a/drivers/infiniband/ulp/ipoib/ipoib_vlan.c > +++ b/drivers/infiniband/ulp/ipoib/ipoib_vlan.c > @@ -58,6 +58,7 @@ int __ipoib_vlan_add(struct ipoib_dev_priv *ppriv, struct ipoib_dev_priv *priv, > /* MTU will be reset when mcast join happens */ > priv->dev->mtu = IPOIB_UD_MTU(priv->max_ib_mtu); > priv->mcast_mtu = priv->admin_mtu = priv->dev->mtu; > + priv->parent = ppriv->dev; > set_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags); > > result = ipoib_set_dev_features(priv, ppriv->ca); > @@ -84,8 +85,6 @@ int __ipoib_vlan_add(struct ipoib_dev_priv *ppriv, struct ipoib_dev_priv *priv, > goto register_failed; > } > > - priv->parent = ppriv->dev; > - > ipoib_create_debug_files(priv->dev); > > /* RTNL childs don't need proprietary sysfs entries */ ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink 2015-04-14 16:30 ` Erez Shitrit @ 2015-04-15 5:16 ` Honggang LI 0 siblings, 0 replies; 53+ messages in thread From: Honggang LI @ 2015-04-15 5:16 UTC (permalink / raw) To: Erez Shitrit Cc: Roland Dreier, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, kaber-dcUjhNyLwpNeoWH0uzbU5w, davem-fT/PcQaiUtIeIZ0/mPfg9Q, Alex Estrin, Doug Ledford, edumazet-hpIqsD4AKlfQT0dZR+AlfA, Erez Shitrit, nicolas.dichtel-pdR9zngts4EAvxtiuMwx3w, maheshb-hpIqsD4AKlfQT0dZR+AlfA, jbenc-H+wXaHxf7aLQT0dZR+AlfA, ebiederm-aS9lmoZGLiVWk0Htik3J/w, elfring-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f, f.fainelli-Re5JQEeQqe8AvxtiuMwx3w, linux-0h96xk9xTtrk1uMJSBkQmQ, andrew-g2DYL2Zd6BY, sfeldma-Re5JQEeQqe8AvxtiuMwx3w, alexander.h.duyck-ral2JQCrhuEAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA On Tue, Apr 14, 2015 at 07:30:03PM +0300, Erez Shitrit wrote: > > @@ -846,7 +846,10 @@ static int ipoib_get_iflink(const struct net_device *dev) > > { > > struct ipoib_dev_priv *priv = netdev_priv(dev); > > > > - return priv->parent->ifindex; > > + if (priv && priv->parent) > > + return priv->parent->ifindex; > > + else > > + return 0; > This will make parent interface to return 0 instead of its own ifindex. > I would suggest write something like that: > > + /* parent interface */ > + if (!test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags)) > + return dev->ifindex; > + Hi, Erez Sorry for delay of reply. It was about 01:00 in the morning, so I went into bed. And thank you for the suggestion. You are right. After insert some printk statements in the driver, I confirmed it. ---------------------- console log -------------------- ipoib_get_iflink: priv = ffff880275e487c0, priv->parent = (null), priv->flags = 0x20f, dev = ffff880275e48000, dev->name = qib_ib1 qib_ib1, test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags) = 0 qib_ib1, idev->ifindex = 14 ipoib_get_iflink: priv = ffff8802765d27c0, priv->parent = (null), priv->flags = 0x20f, dev = ffff8802765d2000, dev->name = qib_ib2 qib_ib2, test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags) = 0 qib_ib2, idev->ifindex = 15 ipoib_get_iflink: priv = ffff8804741a47c0, priv->parent = ffff880275e48000, priv->flags = 0x224, dev = ffff8804741a4000, dev->name = qib_ib1.8003 qib_ib1.8003, test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags) = 1 qib_ib1.8003, idev->ifindex = 16 ---------------------- console log -------------------- I will rewrite the patch. > + /* child/vlan interface */ > + if (!priv->parent) > + return -1; > + > return priv->parent->ifindex; > > Thanks, > Erez. > > > } > > > > static u32 ipoib_addr_hash(struct ipoib_neigh_hash *htbl, u8 *daddr) > > -- > > 1.8.3.1 > > -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink @ 2015-04-15 5:16 ` Honggang LI 0 siblings, 0 replies; 53+ messages in thread From: Honggang LI @ 2015-04-15 5:16 UTC (permalink / raw) To: Erez Shitrit Cc: Roland Dreier, sean.hefty, hal.rosenstock, kaber, davem, Alex Estrin, Doug Ledford, edumazet, Erez Shitrit, nicolas.dichtel, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev On Tue, Apr 14, 2015 at 07:30:03PM +0300, Erez Shitrit wrote: > > @@ -846,7 +846,10 @@ static int ipoib_get_iflink(const struct net_device *dev) > > { > > struct ipoib_dev_priv *priv = netdev_priv(dev); > > > > - return priv->parent->ifindex; > > + if (priv && priv->parent) > > + return priv->parent->ifindex; > > + else > > + return 0; > This will make parent interface to return 0 instead of its own ifindex. > I would suggest write something like that: > > + /* parent interface */ > + if (!test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags)) > + return dev->ifindex; > + Hi, Erez Sorry for delay of reply. It was about 01:00 in the morning, so I went into bed. And thank you for the suggestion. You are right. After insert some printk statements in the driver, I confirmed it. ---------------------- console log -------------------- ipoib_get_iflink: priv = ffff880275e487c0, priv->parent = (null), priv->flags = 0x20f, dev = ffff880275e48000, dev->name = qib_ib1 qib_ib1, test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags) = 0 qib_ib1, idev->ifindex = 14 ipoib_get_iflink: priv = ffff8802765d27c0, priv->parent = (null), priv->flags = 0x20f, dev = ffff8802765d2000, dev->name = qib_ib2 qib_ib2, test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags) = 0 qib_ib2, idev->ifindex = 15 ipoib_get_iflink: priv = ffff8804741a47c0, priv->parent = ffff880275e48000, priv->flags = 0x224, dev = ffff8804741a4000, dev->name = qib_ib1.8003 qib_ib1.8003, test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags) = 1 qib_ib1.8003, idev->ifindex = 16 ---------------------- console log -------------------- I will rewrite the patch. > + /* child/vlan interface */ > + if (!priv->parent) > + return -1; > + > return priv->parent->ifindex; > > Thanks, > Erez. > > > } > > > > static u32 ipoib_addr_hash(struct ipoib_neigh_hash *htbl, u8 *daddr) > > -- > > 1.8.3.1 > > ^ permalink raw reply [flat|nested] 53+ messages in thread
[parent not found: <20150415051640.GB4881-9l7K0WC0B0wP68cbUhXDDlaTQe2KTcn/@public.gmane.org>]
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink 2015-04-15 5:16 ` Honggang LI @ 2015-04-15 6:57 ` Honggang LI -1 siblings, 0 replies; 53+ messages in thread From: Honggang LI @ 2015-04-15 6:57 UTC (permalink / raw) To: Erez Shitrit Cc: Roland Dreier, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, kaber-dcUjhNyLwpNeoWH0uzbU5w, davem-fT/PcQaiUtIeIZ0/mPfg9Q, Alex Estrin, Doug Ledford, edumazet-hpIqsD4AKlfQT0dZR+AlfA, Erez Shitrit, nicolas.dichtel-pdR9zngts4EAvxtiuMwx3w, maheshb-hpIqsD4AKlfQT0dZR+AlfA, jbenc-H+wXaHxf7aLQT0dZR+AlfA, ebiederm-aS9lmoZGLiVWk0Htik3J/w, elfring-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f, f.fainelli-Re5JQEeQqe8AvxtiuMwx3w, linux-0h96xk9xTtrk1uMJSBkQmQ, andrew-g2DYL2Zd6BY, sfeldma-Re5JQEeQqe8AvxtiuMwx3w, alexander.h.duyck-ral2JQCrhuEAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA There was network accident in the office. I can't find this email in public mailing list. So, I reset it. If you had recived this, please ignore it. thanks On Wed, Apr 15, 2015 at 01:16:40PM +0800, Honggang LI wrote: > On Tue, Apr 14, 2015 at 07:30:03PM +0300, Erez Shitrit wrote: > > > @@ -846,7 +846,10 @@ static int ipoib_get_iflink(const struct net_device *dev) > > > { > > > struct ipoib_dev_priv *priv = netdev_priv(dev); > > > > > > - return priv->parent->ifindex; > > > + if (priv && priv->parent) > > > + return priv->parent->ifindex; > > > + else > > > + return 0; > > This will make parent interface to return 0 instead of its own ifindex. > > I would suggest write something like that: > > > > + /* parent interface */ > > + if (!test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags)) > > + return dev->ifindex; > > + > > Hi, Erez > > Sorry for delay of reply. It was about 01:00 in the morning, so I > went into bed. And thank you for the suggestion. You are right. After > insert some printk statements in the driver, I confirmed it. > > ---------------------- console log -------------------- > ipoib_get_iflink: priv = ffff880275e487c0, priv->parent = (null), priv->flags = 0x20f, dev = ffff880275e48000, dev->name = qib_ib1 > qib_ib1, test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags) = 0 > qib_ib1, idev->ifindex = 14 > > ipoib_get_iflink: priv = ffff8802765d27c0, priv->parent = (null), priv->flags = 0x20f, dev = ffff8802765d2000, dev->name = qib_ib2 > qib_ib2, test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags) = 0 > qib_ib2, idev->ifindex = 15 > > ipoib_get_iflink: priv = ffff8804741a47c0, priv->parent = ffff880275e48000, priv->flags = 0x224, dev = ffff8804741a4000, dev->name = qib_ib1.8003 > qib_ib1.8003, test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags) = 1 > qib_ib1.8003, idev->ifindex = 16 > > ---------------------- console log -------------------- > > I will rewrite the patch. > > > + /* child/vlan interface */ > > + if (!priv->parent) > > + return -1; > > + > > return priv->parent->ifindex; > > > > Thanks, > > Erez. > > > > > } > > > > > > static u32 ipoib_addr_hash(struct ipoib_neigh_hash *htbl, u8 *daddr) > > > -- > > > 1.8.3.1 > > > -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink @ 2015-04-15 6:57 ` Honggang LI 0 siblings, 0 replies; 53+ messages in thread From: Honggang LI @ 2015-04-15 6:57 UTC (permalink / raw) To: Erez Shitrit Cc: Roland Dreier, sean.hefty, hal.rosenstock, kaber, davem, Alex Estrin, Doug Ledford, edumazet, Erez Shitrit, nicolas.dichtel, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev There was network accident in the office. I can't find this email in public mailing list. So, I reset it. If you had recived this, please ignore it. thanks On Wed, Apr 15, 2015 at 01:16:40PM +0800, Honggang LI wrote: > On Tue, Apr 14, 2015 at 07:30:03PM +0300, Erez Shitrit wrote: > > > @@ -846,7 +846,10 @@ static int ipoib_get_iflink(const struct net_device *dev) > > > { > > > struct ipoib_dev_priv *priv = netdev_priv(dev); > > > > > > - return priv->parent->ifindex; > > > + if (priv && priv->parent) > > > + return priv->parent->ifindex; > > > + else > > > + return 0; > > This will make parent interface to return 0 instead of its own ifindex. > > I would suggest write something like that: > > > > + /* parent interface */ > > + if (!test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags)) > > + return dev->ifindex; > > + > > Hi, Erez > > Sorry for delay of reply. It was about 01:00 in the morning, so I > went into bed. And thank you for the suggestion. You are right. After > insert some printk statements in the driver, I confirmed it. > > ---------------------- console log -------------------- > ipoib_get_iflink: priv = ffff880275e487c0, priv->parent = (null), priv->flags = 0x20f, dev = ffff880275e48000, dev->name = qib_ib1 > qib_ib1, test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags) = 0 > qib_ib1, idev->ifindex = 14 > > ipoib_get_iflink: priv = ffff8802765d27c0, priv->parent = (null), priv->flags = 0x20f, dev = ffff8802765d2000, dev->name = qib_ib2 > qib_ib2, test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags) = 0 > qib_ib2, idev->ifindex = 15 > > ipoib_get_iflink: priv = ffff8804741a47c0, priv->parent = ffff880275e48000, priv->flags = 0x224, dev = ffff8804741a4000, dev->name = qib_ib1.8003 > qib_ib1.8003, test_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags) = 1 > qib_ib1.8003, idev->ifindex = 16 > > ---------------------- console log -------------------- > > I will rewrite the patch. > > > + /* child/vlan interface */ > > + if (!priv->parent) > > + return -1; > > + > > return priv->parent->ifindex; > > > > Thanks, > > Erez. > > > > > } > > > > > > static u32 ipoib_addr_hash(struct ipoib_neigh_hash *htbl, u8 *daddr) > > > -- > > > 1.8.3.1 > > > ^ permalink raw reply [flat|nested] 53+ messages in thread
* [PATCH linux-next 2/4] ipvlan: fix possible NULL pointer dereference in ipvlan_get_iflink 2015-04-14 15:20 ` Honggang Li (?) (?) @ 2015-04-14 15:20 ` Honggang Li -1 siblings, 0 replies; 53+ messages in thread From: Honggang Li @ 2015-04-14 15:20 UTC (permalink / raw) To: roland, sean.hefty, hal.rosenstock, kaber, davem, alex.estrin, dledford, edumazet, erezsh, nicolas.dichtel, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev Cc: Honggang Li Signed-off-by: Honggang Li <honli@redhat.com> --- drivers/net/ipvlan/ipvlan_main.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/net/ipvlan/ipvlan_main.c b/drivers/net/ipvlan/ipvlan_main.c index 77b92a0..479fcf7 100644 --- a/drivers/net/ipvlan/ipvlan_main.c +++ b/drivers/net/ipvlan/ipvlan_main.c @@ -308,7 +308,10 @@ static int ipvlan_get_iflink(const struct net_device *dev) { struct ipvl_dev *ipvlan = netdev_priv(dev); - return ipvlan->phy_dev->ifindex; + if (ipvlan && pvlan->phy_dev) + return ipvlan->phy_dev->ifindex; + else + return 0; } static const struct net_device_ops ipvlan_netdev_ops = { -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 53+ messages in thread
[parent not found: <1429024817-21561-1-git-send-email-honli-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>]
* [PATCH linux-next 3/4] macvlan: fix possible NULL pointer dereference in macvlan_dev_get_iflink 2015-04-14 15:20 ` Honggang Li @ 2015-04-14 15:20 ` Honggang Li -1 siblings, 0 replies; 53+ messages in thread From: Honggang Li @ 2015-04-14 15:20 UTC (permalink / raw) To: roland-DgEjT+Ai2ygdnm+yROfE0A, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, kaber-dcUjhNyLwpNeoWH0uzbU5w, davem-fT/PcQaiUtIeIZ0/mPfg9Q, alex.estrin-ral2JQCrhuEAvxtiuMwx3w, dledford-H+wXaHxf7aLQT0dZR+AlfA, edumazet-hpIqsD4AKlfQT0dZR+AlfA, erezsh-VPRAkNaXOzVWk0Htik3J/w, nicolas.dichtel-pdR9zngts4EAvxtiuMwx3w, maheshb-hpIqsD4AKlfQT0dZR+AlfA, jbenc-H+wXaHxf7aLQT0dZR+AlfA, ebiederm-aS9lmoZGLiVWk0Htik3J/w, elfring-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f, f.fainelli-Re5JQEeQqe8AvxtiuMwx3w, linux-0h96xk9xTtrk1uMJSBkQmQ, andrew-g2DYL2Zd6BY, sfeldma-Re5JQEeQqe8AvxtiuMwx3w, alexander.h.duyck-ral2JQCrhuEAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA Cc: Honggang Li Signed-off-by: Honggang Li <honli-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> --- drivers/net/macvlan.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/net/macvlan.c b/drivers/net/macvlan.c index b227a13..1e59f39 100644 --- a/drivers/net/macvlan.c +++ b/drivers/net/macvlan.c @@ -998,7 +998,9 @@ static int macvlan_dev_get_iflink(const struct net_device *dev) { struct macvlan_dev *vlan = netdev_priv(dev); - return vlan->lowerdev->ifindex; + if (vlan && vlan->lowerdev) + return vlan->lowerdev->ifindex; + return 0; } static const struct ethtool_ops macvlan_ethtool_ops = { -- 1.8.3.1 -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 53+ messages in thread
* [PATCH linux-next 3/4] macvlan: fix possible NULL pointer dereference in macvlan_dev_get_iflink @ 2015-04-14 15:20 ` Honggang Li 0 siblings, 0 replies; 53+ messages in thread From: Honggang Li @ 2015-04-14 15:20 UTC (permalink / raw) To: roland, sean.hefty, hal.rosenstock, kaber, davem, alex.estrin, dledford, edumazet, erezsh, nicolas.dichtel, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev Cc: Honggang Li Signed-off-by: Honggang Li <honli@redhat.com> --- drivers/net/macvlan.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/net/macvlan.c b/drivers/net/macvlan.c index b227a13..1e59f39 100644 --- a/drivers/net/macvlan.c +++ b/drivers/net/macvlan.c @@ -998,7 +998,9 @@ static int macvlan_dev_get_iflink(const struct net_device *dev) { struct macvlan_dev *vlan = netdev_priv(dev); - return vlan->lowerdev->ifindex; + if (vlan && vlan->lowerdev) + return vlan->lowerdev->ifindex; + return 0; } static const struct ethtool_ops macvlan_ethtool_ops = { -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 3/4] macvlan: fix possible NULL pointer dereference in macvlan_dev_get_iflink 2015-04-14 15:20 ` Honggang Li (?) @ 2015-04-14 15:26 ` Patrick McHardy 2015-04-14 15:32 ` Honggang LI 2015-04-14 15:35 ` Nicolas Dichtel -1 siblings, 2 replies; 53+ messages in thread From: Patrick McHardy @ 2015-04-14 15:26 UTC (permalink / raw) To: Honggang Li Cc: roland, sean.hefty, hal.rosenstock, davem, alex.estrin, dledford, edumazet, erezsh, nicolas.dichtel, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev On 14.04, Honggang Li wrote: > Signed-off-by: Honggang Li <honli@redhat.com> > --- > drivers/net/macvlan.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/macvlan.c b/drivers/net/macvlan.c > index b227a13..1e59f39 100644 > --- a/drivers/net/macvlan.c > +++ b/drivers/net/macvlan.c > @@ -998,7 +998,9 @@ static int macvlan_dev_get_iflink(const struct net_device *dev) > { > struct macvlan_dev *vlan = netdev_priv(dev); > > - return vlan->lowerdev->ifindex; > + if (vlan && vlan->lowerdev) > + return vlan->lowerdev->ifindex; That is completely useless. vlan (=netdev_priv) can not be NULL as netdev_priv() never returns NULL and vlan->lowerdev is always valid because a macvlan wouldn't make much sense otherwise. ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 3/4] macvlan: fix possible NULL pointer dereference in macvlan_dev_get_iflink 2015-04-14 15:26 ` Patrick McHardy @ 2015-04-14 15:32 ` Honggang LI 2015-04-14 15:35 ` Patrick McHardy 2015-04-14 17:47 ` David Miller 2015-04-14 15:35 ` Nicolas Dichtel 1 sibling, 2 replies; 53+ messages in thread From: Honggang LI @ 2015-04-14 15:32 UTC (permalink / raw) To: Patrick McHardy Cc: roland, sean.hefty, hal.rosenstock, davem, alex.estrin, dledford, edumazet, erezsh, nicolas.dichtel, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev On Tue, Apr 14, 2015 at 04:26:27PM +0100, Patrick McHardy wrote: > > That is completely useless. vlan (=netdev_priv) can not be NULL as > netdev_priv() never returns NULL and vlan->lowerdev is always valid > because a macvlan wouldn't make much sense otherwise. OK, please drop this patch. thanks ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 3/4] macvlan: fix possible NULL pointer dereference in macvlan_dev_get_iflink 2015-04-14 15:32 ` Honggang LI @ 2015-04-14 15:35 ` Patrick McHardy 2015-04-14 17:47 ` David Miller 1 sibling, 0 replies; 53+ messages in thread From: Patrick McHardy @ 2015-04-14 15:35 UTC (permalink / raw) To: Honggang LI Cc: roland, sean.hefty, hal.rosenstock, davem, alex.estrin, dledford, edumazet, erezsh, nicolas.dichtel, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev On 14.04, Honggang LI wrote: > On Tue, Apr 14, 2015 at 04:26:27PM +0100, Patrick McHardy wrote: > > > > That is completely useless. vlan (=netdev_priv) can not be NULL as > > netdev_priv() never returns NULL and vlan->lowerdev is always valid > > because a macvlan wouldn't make much sense otherwise. > > OK, please drop this patch. Well, the fact that netdev_priv never return NULL implies all your patches need to be redone. And I suggest you check whether this condition can actually happen. Just because it can in one driver says nothing at all about others. ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 3/4] macvlan: fix possible NULL pointer dereference in macvlan_dev_get_iflink 2015-04-14 15:32 ` Honggang LI 2015-04-14 15:35 ` Patrick McHardy @ 2015-04-14 17:47 ` David Miller 1 sibling, 0 replies; 53+ messages in thread From: David Miller @ 2015-04-14 17:47 UTC (permalink / raw) To: honli Cc: kaber, roland, sean.hefty, hal.rosenstock, alex.estrin, dledford, edumazet, erezsh, nicolas.dichtel, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev From: Honggang LI <honli@redhat.com> Date: Tue, 14 Apr 2015 23:32:39 +0800 > On Tue, Apr 14, 2015 at 04:26:27PM +0100, Patrick McHardy wrote: >> >> That is completely useless. vlan (=netdev_priv) can not be NULL as >> netdev_priv() never returns NULL and vlan->lowerdev is always valid >> because a macvlan wouldn't make much sense otherwise. > > OK, please drop this patch. That's not how this works. When a patch series needs any chnages, you must make a fresh, new complete submission of the entire patch series. ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 3/4] macvlan: fix possible NULL pointer dereference in macvlan_dev_get_iflink 2015-04-14 15:26 ` Patrick McHardy 2015-04-14 15:32 ` Honggang LI @ 2015-04-14 15:35 ` Nicolas Dichtel [not found] ` <552D33B0.6040808-pdR9zngts4EAvxtiuMwx3w@public.gmane.org> 1 sibling, 1 reply; 53+ messages in thread From: Nicolas Dichtel @ 2015-04-14 15:35 UTC (permalink / raw) To: Patrick McHardy, Honggang Li Cc: roland, sean.hefty, hal.rosenstock, davem, alex.estrin, dledford, edumazet, erezsh, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev Le 14/04/2015 17:26, Patrick McHardy a écrit : > On 14.04, Honggang Li wrote: [snip] >> - return vlan->lowerdev->ifindex; >> + if (vlan && vlan->lowerdev) >> + return vlan->lowerdev->ifindex; > > That is completely useless. vlan (=netdev_priv) can not be NULL as > netdev_priv() never returns NULL and vlan->lowerdev is always valid > because a macvlan wouldn't make much sense otherwise. > And I suspect that it is the same for ipvlan and dsa. ^ permalink raw reply [flat|nested] 53+ messages in thread
[parent not found: <552D33B0.6040808-pdR9zngts4EAvxtiuMwx3w@public.gmane.org>]
* Re: [PATCH linux-next 3/4] macvlan: fix possible NULL pointer dereference in macvlan_dev_get_iflink 2015-04-14 15:35 ` Nicolas Dichtel @ 2015-04-14 15:37 ` Andrew Lunn 0 siblings, 0 replies; 53+ messages in thread From: Andrew Lunn @ 2015-04-14 15:37 UTC (permalink / raw) To: Nicolas Dichtel Cc: Patrick McHardy, Honggang Li, roland-DgEjT+Ai2ygdnm+yROfE0A, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, davem-fT/PcQaiUtIeIZ0/mPfg9Q, alex.estrin-ral2JQCrhuEAvxtiuMwx3w, dledford-H+wXaHxf7aLQT0dZR+AlfA, edumazet-hpIqsD4AKlfQT0dZR+AlfA, erezsh-VPRAkNaXOzVWk0Htik3J/w, maheshb-hpIqsD4AKlfQT0dZR+AlfA, jbenc-H+wXaHxf7aLQT0dZR+AlfA, ebiederm-aS9lmoZGLiVWk0Htik3J/w, elfring-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f, f.fainelli-Re5JQEeQqe8AvxtiuMwx3w, linux-0h96xk9xTtrk1uMJSBkQmQ, sfeldma-Re5JQEeQqe8AvxtiuMwx3w, alexander.h.duyck-ral2JQCrhuEAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA On Tue, Apr 14, 2015 at 05:35:12PM +0200, Nicolas Dichtel wrote: > Le 14/04/2015 17:26, Patrick McHardy a écrit : > >On 14.04, Honggang Li wrote: > [snip] > >>- return vlan->lowerdev->ifindex; > >>+ if (vlan && vlan->lowerdev) > >>+ return vlan->lowerdev->ifindex; > > > >That is completely useless. vlan (=netdev_priv) can not be NULL as > >netdev_priv() never returns NULL and vlan->lowerdev is always valid > >because a macvlan wouldn't make much sense otherwise. > > > And I suspect that it is the same for ipvlan and dsa. I agree about DSA. I don't see any way this could happen. Andrew -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 3/4] macvlan: fix possible NULL pointer dereference in macvlan_dev_get_iflink @ 2015-04-14 15:37 ` Andrew Lunn 0 siblings, 0 replies; 53+ messages in thread From: Andrew Lunn @ 2015-04-14 15:37 UTC (permalink / raw) To: Nicolas Dichtel Cc: Patrick McHardy, Honggang Li, roland, sean.hefty, hal.rosenstock, davem, alex.estrin, dledford, edumazet, erezsh, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev On Tue, Apr 14, 2015 at 05:35:12PM +0200, Nicolas Dichtel wrote: > Le 14/04/2015 17:26, Patrick McHardy a écrit : > >On 14.04, Honggang Li wrote: > [snip] > >>- return vlan->lowerdev->ifindex; > >>+ if (vlan && vlan->lowerdev) > >>+ return vlan->lowerdev->ifindex; > > > >That is completely useless. vlan (=netdev_priv) can not be NULL as > >netdev_priv() never returns NULL and vlan->lowerdev is always valid > >because a macvlan wouldn't make much sense otherwise. > > > And I suspect that it is the same for ipvlan and dsa. I agree about DSA. I don't see any way this could happen. Andrew ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 3/4] macvlan: fix possible NULL pointer dereference in macvlan_dev_get_iflink 2015-04-14 15:37 ` Andrew Lunn (?) @ 2015-04-14 15:46 ` Honggang LI -1 siblings, 0 replies; 53+ messages in thread From: Honggang LI @ 2015-04-14 15:46 UTC (permalink / raw) To: Andrew Lunn Cc: Nicolas Dichtel, Patrick McHardy, roland, sean.hefty, hal.rosenstock, davem, alex.estrin, dledford, edumazet, erezsh, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev On Tue, Apr 14, 2015 at 05:37:57PM +0200, Andrew Lunn wrote: > > > > > And I suspect that it is the same for ipvlan and dsa. > > I agree about DSA. I don't see any way this could happen. > > Andrew I only keep the ipoib patch and drop the rest patches. thanks ^ permalink raw reply [flat|nested] 53+ messages in thread
* [PATCH linux-next v2] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink 2015-04-14 15:20 ` Honggang Li @ 2015-04-14 16:26 ` Honggang Li -1 siblings, 0 replies; 53+ messages in thread From: Honggang Li @ 2015-04-14 16:26 UTC (permalink / raw) To: roland-DgEjT+Ai2ygdnm+yROfE0A, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, davem-fT/PcQaiUtIeIZ0/mPfg9Q, alex.estrin-ral2JQCrhuEAvxtiuMwx3w, dledford-H+wXaHxf7aLQT0dZR+AlfA, edumazet-hpIqsD4AKlfQT0dZR+AlfA, erezsh-VPRAkNaXOzVWk0Htik3J/w, nicolas.dichtel-pdR9zngts4EAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA Cc: Honggang Li 1) Droped ipvlan/macvlan/dsa patches. 2) Do not check the pointer returned by netdev_priv. 3) Add the FIX tag. --------------------- console log --------------- ipoib_get_iflink dev = ffff88026eb1c000, priv = ffff88026eb1c7c0, priv->parent = (null) ipoib_get_iflink dev = ffff88026eb1c000, priv = ffff88026eb1c7c0, priv->parent = (null) ipoib_get_iflink dev = ffff880225c8a000, priv = ffff880225c8a7c0, priv->parent = (null) ipoib_get_iflink dev = ffff880225c8a000, priv = ffff880225c8a7c0, priv->parent = (null) [ OK ] ipoib_get_iflink dev = ffff88026eb1c000, priv = ffff88026eb1c7c0, priv->parent = (null) ipoib_get_iflink dev = ffff88026eb1c000, priv = ffff88026eb1c7c0, priv->parent = (null) ipoib_get_iflink dev = ffff880225c8a000, priv = ffff880225c8a7c0, priv->parent = (null) ipoib_get_iflink dev = ffff880225c8a000, priv = ffff880225c8a7c0, priv->parent = (null) Shutting down interface cxgb4_1: ipoib_get_iflink dev = ffff88026eb1c000, priv = ffff88026eb1c7c0, priv->parent = (null) ipoib_get_iflink dev = ffff88026eb1c000, priv = ffff88026eb1c7c0, priv->parent = (null) ipoib_get_iflink dev = ffff880225c8a000, priv = ffff880225c8a7c0, priv->parent = (null) ipoib_get_iflink dev = ffff880225c8a000, priv = ffff880225c8a7c0, priv->parent = (null) --------------------- console log --------------- Honggang Li (1): infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink drivers/infiniband/ulp/ipoib/ipoib_main.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) -- 1.8.3.1 -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 53+ messages in thread
* [PATCH linux-next v2] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink @ 2015-04-14 16:26 ` Honggang Li 0 siblings, 0 replies; 53+ messages in thread From: Honggang Li @ 2015-04-14 16:26 UTC (permalink / raw) To: roland, sean.hefty, hal.rosenstock, davem, alex.estrin, dledford, edumazet, erezsh, nicolas.dichtel, linux-rdma, linux-kernel Cc: Honggang Li 1) Droped ipvlan/macvlan/dsa patches. 2) Do not check the pointer returned by netdev_priv. 3) Add the FIX tag. --------------------- console log --------------- ipoib_get_iflink dev = ffff88026eb1c000, priv = ffff88026eb1c7c0, priv->parent = (null) ipoib_get_iflink dev = ffff88026eb1c000, priv = ffff88026eb1c7c0, priv->parent = (null) ipoib_get_iflink dev = ffff880225c8a000, priv = ffff880225c8a7c0, priv->parent = (null) ipoib_get_iflink dev = ffff880225c8a000, priv = ffff880225c8a7c0, priv->parent = (null) [ OK ] ipoib_get_iflink dev = ffff88026eb1c000, priv = ffff88026eb1c7c0, priv->parent = (null) ipoib_get_iflink dev = ffff88026eb1c000, priv = ffff88026eb1c7c0, priv->parent = (null) ipoib_get_iflink dev = ffff880225c8a000, priv = ffff880225c8a7c0, priv->parent = (null) ipoib_get_iflink dev = ffff880225c8a000, priv = ffff880225c8a7c0, priv->parent = (null) Shutting down interface cxgb4_1: ipoib_get_iflink dev = ffff88026eb1c000, priv = ffff88026eb1c7c0, priv->parent = (null) ipoib_get_iflink dev = ffff88026eb1c000, priv = ffff88026eb1c7c0, priv->parent = (null) ipoib_get_iflink dev = ffff880225c8a000, priv = ffff880225c8a7c0, priv->parent = (null) ipoib_get_iflink dev = ffff880225c8a000, priv = ffff880225c8a7c0, priv->parent = (null) --------------------- console log --------------- Honggang Li (1): infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink drivers/infiniband/ulp/ipoib/ipoib_main.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) -- 1.8.3.1 ^ permalink raw reply [flat|nested] 53+ messages in thread
[parent not found: <1429028811-29888-1-git-send-email-honli-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>]
* [PATCH] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink 2015-04-14 16:26 ` Honggang Li @ 2015-04-14 16:26 ` Honggang Li -1 siblings, 0 replies; 53+ messages in thread From: Honggang Li @ 2015-04-14 16:26 UTC (permalink / raw) To: roland-DgEjT+Ai2ygdnm+yROfE0A, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, davem-fT/PcQaiUtIeIZ0/mPfg9Q, alex.estrin-ral2JQCrhuEAvxtiuMwx3w, dledford-H+wXaHxf7aLQT0dZR+AlfA, edumazet-hpIqsD4AKlfQT0dZR+AlfA, erezsh-VPRAkNaXOzVWk0Htik3J/w, nicolas.dichtel-pdR9zngts4EAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA Cc: Honggang Li Starting monitoring for VG vg_rdma01: 3 logical volume(s) in volume group "vg_rdma01" monitored [ OK ] Starting cgconfig service: Failed to parse /etc/cgconfig.conf or /etc/cgconfig.d[FAILED] Loading OpenIB kernel modules: BUG: unable to handle kernel NULL pointer dereference at 0000000000000120 IP: [<ffffffffa06b9060>] ipoib_get_iflink+0x10/0x20 [ib_ipoib] PGD 475540067 PUD 473541067 PMD 0 Oops: 0000 [#1] SMP Modules linked in: ib_ipoib(+) rdma_ucm ib_ucm ib_uverbs ib_umad rdma_cm ib_cm ib_sa vhost_net macvtap macvlan vhost tun ipmi_devintf sg ipmi_si ipmi_msghandler serio_raw iTCO_wdt iTCO_vendor_support cdc_ether usbnet mii bnx2 intel_powerclamp coretemp kvm_intel kvm crc32c_intel ghash_clmulni_intel aesni_intel ablk_helper cryptd lrw gf128mul glue_helper aes_x86_64 microcode pcspkr i2c_i801 i2c_core lpc_ich mfd_core acpi_cpufreq ioatdma i7core_edac edac_core shpchp ext4(E) jbd2(E) mbcache(E) sd_mod(E) megaraid_sas(E) pata_acpi(E) ata_generic(E) ata_piix(E) iw_cxgb3(E) cxgb3(E) mdio(E) ib_qib(E) dca(E) ib_mad(E) iw_cxgb4(E) iw_cm(E) ib_core(E) ib_addr(E) ipv6(E) cxgb4(E) dm_mirror(E) dm_region_hash(E) dm_log(E) dm_mod(E) CPU: 6 PID: 2405 Comm: modprobe Tainted: G E 4.0.0-next-20150413 #1 Hardware name: IBM System x3650 M3 -[7945O63]-/00D4062, BIOS -[D6E157AUS-1.15]- 06/13/2012 task: ffff880476ad6f00 ti: ffff88047579c000 task.ti: ffff88047579c000 RIP: 0010:[<ffffffffa06b9060>] [<ffffffffa06b9060>] ipoib_get_iflink+0x10/0x20 [ib_ipoib] RSP: 0018:ffff88047579f9b8 EFLAGS: 00010286 RAX: 0000000000000000 RBX: ffff880476e2a000 RCX: 0000000000000000 RDX: 0000000000000004 RSI: ffff88047579fbb8 RDI: ffff880476e2a000 RBP: ffff88047579f9b8 R08: 0000000000000660 R09: ffff88047404f068 R10: 0000000000000000 R11: 0000000000000000 R12: ffff8804736bec00 R13: ffff88047579fbb4 R14: ffff88047404f000 R15: 0000000000000009 FS: 00007fc047a2e700(0000) GS:ffff88047fc00000(0000) knlGS:0000000000000000 CS: 0010 DS: 0000 ES: 0000 CR0: 000000008005003b CR2: 0000000000000120 CR3: 000000047541f000 CR4: 00000000000006e0 Stack: ffff88047579f9c8 ffffffff814fbfa3 ffff88047579fbe8 ffffffff81515a15 0000000000000005 ffff880476e2a280 0000000000000005 0000000000000014 ffff88047579fa48 ffffffff8150a577 0000000000000000 ffff8804ffffffff Call Trace: [<ffffffff814fbfa3>] dev_get_iflink+0x23/0x40 [<ffffffff81515a15>] rtnl_fill_ifinfo+0x255/0xce0 [<ffffffff8150a577>] ? __hw_addr_create_ex+0x97/0xc0 [<ffffffff815d32bb>] ? _raw_spin_unlock_bh+0x1b/0x20 [<ffffffff8150a8e5>] ? __dev_mc_add+0x75/0x90 [<ffffffffa00a115c>] ? igmp6_group_added+0x5c/0x130 [ipv6] [<ffffffff8119c6cc>] ? __kmalloc_node_track_caller+0x3c/0x50 [<ffffffff814f0f0b>] ? __kmalloc_reserve+0x3b/0xa0 [<ffffffff814f12f8>] ? __alloc_skb+0xa8/0x1f0 [<ffffffff81516783>] rtmsg_ifinfo_build_skb+0x83/0xe0 [<ffffffff81078fa6>] ? raw_notifier_call_chain+0x16/0x20 [<ffffffff81516801>] rtmsg_ifinfo+0x21/0x40 [<ffffffff81504eaf>] register_netdevice+0x38f/0x400 [<ffffffff81504f3e>] register_netdev+0x1e/0x30 [<ffffffffa06bc204>] ipoib_add_port.clone.0+0x214/0x390 [ib_ipoib] [<ffffffffa06bc447>] ipoib_add_one+0xc7/0x110 [ib_ipoib] [<ffffffffa00f9d4d>] ib_register_client+0x7d/0xa0 [ib_core] [<ffffffffa06ce000>] ? 0xffffffffa06ce000 [<ffffffffa06ce0f2>] ipoib_init_module+0xf2/0x13c [ib_ipoib] [<ffffffff81000287>] do_one_initcall+0xb7/0x1d0 [<ffffffff810d8189>] do_init_module+0x69/0x200 [<ffffffff810da985>] load_module+0x5b5/0x730 [<ffffffff810d79b0>] ? mod_sysfs_teardown+0x150/0x150 [<ffffffff81183232>] ? __vmalloc+0x22/0x30 [<ffffffff810d73c0>] ? module_sect_show+0x30/0x30 [<ffffffff810dac84>] SyS_init_module+0x94/0xc0 [<ffffffff815d3997>] system_call_fastpath+0x12/0x6a Code: 66 66 66 90 b9 1e 00 00 00 48 89 f0 48 8d 77 08 48 89 c7 f3 48 a5 c9 c3 0f 1f 00 55 48 89 e5 66 66 66 66 90 48 8b 87 e8 13 00 00 <8b> 80 20 01 00 00 c9 c3 0f 1f 84 00 00 00 00 00 55 48 89 e5 66 RIP [<ffffffffa06b9060>] ipoib_get_iflink+0x10/0x20 [ib_ipoib] RSP <ffff88047579f9b8> CR2: 0000000000000120 ---[ end trace a8610f6e9640eb85 ]--- Fixes: 5aa7add8f14b ("infiniband/ipoib: implement ndo_get_iflink") Signed-off-by: Honggang Li <honli-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> --- drivers/infiniband/ulp/ipoib/ipoib_main.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/infiniband/ulp/ipoib/ipoib_main.c b/drivers/infiniband/ulp/ipoib/ipoib_main.c index 657b89b..fb2a9df 100644 --- a/drivers/infiniband/ulp/ipoib/ipoib_main.c +++ b/drivers/infiniband/ulp/ipoib/ipoib_main.c @@ -846,7 +846,10 @@ static int ipoib_get_iflink(const struct net_device *dev) { struct ipoib_dev_priv *priv = netdev_priv(dev); - return priv->parent->ifindex; + if (priv->parent) + return priv->parent->ifindex; + else + return 0; } static u32 ipoib_addr_hash(struct ipoib_neigh_hash *htbl, u8 *daddr) -- 1.8.3.1 -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 53+ messages in thread
* [PATCH] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink @ 2015-04-14 16:26 ` Honggang Li 0 siblings, 0 replies; 53+ messages in thread From: Honggang Li @ 2015-04-14 16:26 UTC (permalink / raw) To: roland, sean.hefty, hal.rosenstock, davem, alex.estrin, dledford, edumazet, erezsh, nicolas.dichtel, linux-rdma, linux-kernel Cc: Honggang Li Starting monitoring for VG vg_rdma01: 3 logical volume(s) in volume group "vg_rdma01" monitored [ OK ] Starting cgconfig service: Failed to parse /etc/cgconfig.conf or /etc/cgconfig.d[FAILED] Loading OpenIB kernel modules: BUG: unable to handle kernel NULL pointer dereference at 0000000000000120 IP: [<ffffffffa06b9060>] ipoib_get_iflink+0x10/0x20 [ib_ipoib] PGD 475540067 PUD 473541067 PMD 0 Oops: 0000 [#1] SMP Modules linked in: ib_ipoib(+) rdma_ucm ib_ucm ib_uverbs ib_umad rdma_cm ib_cm ib_sa vhost_net macvtap macvlan vhost tun ipmi_devintf sg ipmi_si ipmi_msghandler serio_raw iTCO_wdt iTCO_vendor_support cdc_ether usbnet mii bnx2 intel_powerclamp coretemp kvm_intel kvm crc32c_intel ghash_clmulni_intel aesni_intel ablk_helper cryptd lrw gf128mul glue_helper aes_x86_64 microcode pcspkr i2c_i801 i2c_core lpc_ich mfd_core acpi_cpufreq ioatdma i7core_edac edac_core shpchp ext4(E) jbd2(E) mbcache(E) sd_mod(E) megaraid_sas(E) pata_acpi(E) ata_generic(E) ata_piix(E) iw_cxgb3(E) cxgb3(E) mdio(E) ib_qib(E) dca(E) ib_mad(E) iw_cxgb4(E) iw_cm(E) ib_core(E) ib_addr(E) ipv6(E) cxgb4(E) dm_mirror(E) dm_region_hash(E) dm_log(E) dm_mod(E) CPU: 6 PID: 2405 Comm: modprobe Tainted: G E 4.0.0-next-20150413 #1 Hardware name: IBM System x3650 M3 -[7945O63]-/00D4062, BIOS -[D6E157AUS-1.15]- 06/13/2012 task: ffff880476ad6f00 ti: ffff88047579c000 task.ti: ffff88047579c000 RIP: 0010:[<ffffffffa06b9060>] [<ffffffffa06b9060>] ipoib_get_iflink+0x10/0x20 [ib_ipoib] RSP: 0018:ffff88047579f9b8 EFLAGS: 00010286 RAX: 0000000000000000 RBX: ffff880476e2a000 RCX: 0000000000000000 RDX: 0000000000000004 RSI: ffff88047579fbb8 RDI: ffff880476e2a000 RBP: ffff88047579f9b8 R08: 0000000000000660 R09: ffff88047404f068 R10: 0000000000000000 R11: 0000000000000000 R12: ffff8804736bec00 R13: ffff88047579fbb4 R14: ffff88047404f000 R15: 0000000000000009 FS: 00007fc047a2e700(0000) GS:ffff88047fc00000(0000) knlGS:0000000000000000 CS: 0010 DS: 0000 ES: 0000 CR0: 000000008005003b CR2: 0000000000000120 CR3: 000000047541f000 CR4: 00000000000006e0 Stack: ffff88047579f9c8 ffffffff814fbfa3 ffff88047579fbe8 ffffffff81515a15 0000000000000005 ffff880476e2a280 0000000000000005 0000000000000014 ffff88047579fa48 ffffffff8150a577 0000000000000000 ffff8804ffffffff Call Trace: [<ffffffff814fbfa3>] dev_get_iflink+0x23/0x40 [<ffffffff81515a15>] rtnl_fill_ifinfo+0x255/0xce0 [<ffffffff8150a577>] ? __hw_addr_create_ex+0x97/0xc0 [<ffffffff815d32bb>] ? _raw_spin_unlock_bh+0x1b/0x20 [<ffffffff8150a8e5>] ? __dev_mc_add+0x75/0x90 [<ffffffffa00a115c>] ? igmp6_group_added+0x5c/0x130 [ipv6] [<ffffffff8119c6cc>] ? __kmalloc_node_track_caller+0x3c/0x50 [<ffffffff814f0f0b>] ? __kmalloc_reserve+0x3b/0xa0 [<ffffffff814f12f8>] ? __alloc_skb+0xa8/0x1f0 [<ffffffff81516783>] rtmsg_ifinfo_build_skb+0x83/0xe0 [<ffffffff81078fa6>] ? raw_notifier_call_chain+0x16/0x20 [<ffffffff81516801>] rtmsg_ifinfo+0x21/0x40 [<ffffffff81504eaf>] register_netdevice+0x38f/0x400 [<ffffffff81504f3e>] register_netdev+0x1e/0x30 [<ffffffffa06bc204>] ipoib_add_port.clone.0+0x214/0x390 [ib_ipoib] [<ffffffffa06bc447>] ipoib_add_one+0xc7/0x110 [ib_ipoib] [<ffffffffa00f9d4d>] ib_register_client+0x7d/0xa0 [ib_core] [<ffffffffa06ce000>] ? 0xffffffffa06ce000 [<ffffffffa06ce0f2>] ipoib_init_module+0xf2/0x13c [ib_ipoib] [<ffffffff81000287>] do_one_initcall+0xb7/0x1d0 [<ffffffff810d8189>] do_init_module+0x69/0x200 [<ffffffff810da985>] load_module+0x5b5/0x730 [<ffffffff810d79b0>] ? mod_sysfs_teardown+0x150/0x150 [<ffffffff81183232>] ? __vmalloc+0x22/0x30 [<ffffffff810d73c0>] ? module_sect_show+0x30/0x30 [<ffffffff810dac84>] SyS_init_module+0x94/0xc0 [<ffffffff815d3997>] system_call_fastpath+0x12/0x6a Code: 66 66 66 90 b9 1e 00 00 00 48 89 f0 48 8d 77 08 48 89 c7 f3 48 a5 c9 c3 0f 1f 00 55 48 89 e5 66 66 66 66 90 48 8b 87 e8 13 00 00 <8b> 80 20 01 00 00 c9 c3 0f 1f 84 00 00 00 00 00 55 48 89 e5 66 RIP [<ffffffffa06b9060>] ipoib_get_iflink+0x10/0x20 [ib_ipoib] RSP <ffff88047579f9b8> CR2: 0000000000000120 ---[ end trace a8610f6e9640eb85 ]--- Fixes: 5aa7add8f14b ("infiniband/ipoib: implement ndo_get_iflink") Signed-off-by: Honggang Li <honli@redhat.com> --- drivers/infiniband/ulp/ipoib/ipoib_main.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/infiniband/ulp/ipoib/ipoib_main.c b/drivers/infiniband/ulp/ipoib/ipoib_main.c index 657b89b..fb2a9df 100644 --- a/drivers/infiniband/ulp/ipoib/ipoib_main.c +++ b/drivers/infiniband/ulp/ipoib/ipoib_main.c @@ -846,7 +846,10 @@ static int ipoib_get_iflink(const struct net_device *dev) { struct ipoib_dev_priv *priv = netdev_priv(dev); - return priv->parent->ifindex; + if (priv->parent) + return priv->parent->ifindex; + else + return 0; } static u32 ipoib_addr_hash(struct ipoib_neigh_hash *htbl, u8 *daddr) -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 53+ messages in thread
* [PATCH linux-next 4/4] net/dsa: fix possible NULL pointer dereference in dsa_slave_get_iflink 2015-04-14 15:20 ` Honggang Li ` (3 preceding siblings ...) (?) @ 2015-04-14 15:20 ` Honggang Li [not found] ` <1429024817-21561-5-git-send-email-honli-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> -1 siblings, 1 reply; 53+ messages in thread From: Honggang Li @ 2015-04-14 15:20 UTC (permalink / raw) To: roland, sean.hefty, hal.rosenstock, kaber, davem, alex.estrin, dledford, edumazet, erezsh, nicolas.dichtel, maheshb, jbenc, ebiederm, elfring, f.fainelli, linux, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev Cc: Honggang Li Signed-off-by: Honggang Li <honli@redhat.com> --- net/dsa/slave.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/net/dsa/slave.c b/net/dsa/slave.c index 827cda56..070b599 100644 --- a/net/dsa/slave.c +++ b/net/dsa/slave.c @@ -59,7 +59,10 @@ static int dsa_slave_get_iflink(const struct net_device *dev) { struct dsa_slave_priv *p = netdev_priv(dev); - return p->parent->dst->master_netdev->ifindex; + if (p && p->parent && p->parent->dst && p->parent->dst->master_netdev) + return p->parent->dst->master_netdev->ifindex; + else + return 0; } static inline bool dsa_port_is_bridged(struct dsa_slave_priv *p) -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 53+ messages in thread
[parent not found: <1429024817-21561-5-git-send-email-honli-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>]
* Re: [PATCH linux-next 4/4] net/dsa: fix possible NULL pointer dereference in dsa_slave_get_iflink 2015-04-14 15:20 ` [PATCH linux-next 4/4] net/dsa: fix possible NULL pointer dereference in dsa_slave_get_iflink Honggang Li @ 2015-04-14 15:55 ` Guenter Roeck 0 siblings, 0 replies; 53+ messages in thread From: Guenter Roeck @ 2015-04-14 15:55 UTC (permalink / raw) To: Honggang Li Cc: roland-DgEjT+Ai2ygdnm+yROfE0A, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, kaber-dcUjhNyLwpNeoWH0uzbU5w, davem-fT/PcQaiUtIeIZ0/mPfg9Q, alex.estrin-ral2JQCrhuEAvxtiuMwx3w, dledford-H+wXaHxf7aLQT0dZR+AlfA, edumazet-hpIqsD4AKlfQT0dZR+AlfA, erezsh-VPRAkNaXOzVWk0Htik3J/w, nicolas.dichtel-pdR9zngts4EAvxtiuMwx3w, maheshb-hpIqsD4AKlfQT0dZR+AlfA, jbenc-H+wXaHxf7aLQT0dZR+AlfA, ebiederm-aS9lmoZGLiVWk0Htik3J/w, elfring-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f, f.fainelli-Re5JQEeQqe8AvxtiuMwx3w, andrew-g2DYL2Zd6BY, sfeldma-Re5JQEeQqe8AvxtiuMwx3w, alexander.h.duyck-ral2JQCrhuEAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA On Tue, Apr 14, 2015 at 11:20:17PM +0800, Honggang Li wrote: > Signed-off-by: Honggang Li <honli-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> > --- > net/dsa/slave.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/net/dsa/slave.c b/net/dsa/slave.c > index 827cda56..070b599 100644 > --- a/net/dsa/slave.c > +++ b/net/dsa/slave.c > @@ -59,7 +59,10 @@ static int dsa_slave_get_iflink(const struct net_device *dev) > { > struct dsa_slave_priv *p = netdev_priv(dev); > > - return p->parent->dst->master_netdev->ifindex; > + if (p && p->parent && p->parent->dst && p->parent->dst->master_netdev) > + return p->parent->dst->master_netdev->ifindex; > + else > + return 0; > } I would expect some explanation why you believe that any of those pointers can be NULL. Guenter -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 53+ messages in thread
* Re: [PATCH linux-next 4/4] net/dsa: fix possible NULL pointer dereference in dsa_slave_get_iflink @ 2015-04-14 15:55 ` Guenter Roeck 0 siblings, 0 replies; 53+ messages in thread From: Guenter Roeck @ 2015-04-14 15:55 UTC (permalink / raw) To: Honggang Li Cc: roland, sean.hefty, hal.rosenstock, kaber, davem, alex.estrin, dledford, edumazet, erezsh, nicolas.dichtel, maheshb, jbenc, ebiederm, elfring, f.fainelli, andrew, sfeldma, alexander.h.duyck, linux-rdma, linux-kernel, netdev On Tue, Apr 14, 2015 at 11:20:17PM +0800, Honggang Li wrote: > Signed-off-by: Honggang Li <honli@redhat.com> > --- > net/dsa/slave.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/net/dsa/slave.c b/net/dsa/slave.c > index 827cda56..070b599 100644 > --- a/net/dsa/slave.c > +++ b/net/dsa/slave.c > @@ -59,7 +59,10 @@ static int dsa_slave_get_iflink(const struct net_device *dev) > { > struct dsa_slave_priv *p = netdev_priv(dev); > > - return p->parent->dst->master_netdev->ifindex; > + if (p && p->parent && p->parent->dst && p->parent->dst->master_netdev) > + return p->parent->dst->master_netdev->ifindex; > + else > + return 0; > } I would expect some explanation why you believe that any of those pointers can be NULL. Guenter ^ permalink raw reply [flat|nested] 53+ messages in thread
end of thread, other threads:[~2015-04-16 11:28 UTC | newest] Thread overview: 53+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2015-04-14 15:20 [PATCH linux-next 0/4] fix possile NULL pointer dereference in ndo_get_iflink callback functions Honggang Li 2015-04-14 15:20 ` Honggang Li 2015-04-14 15:20 ` [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink Honggang Li [not found] ` <1429024817-21561-2-git-send-email-honli-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2015-04-14 15:34 ` Eric Dumazet 2015-04-14 15:34 ` Eric Dumazet [not found] ` <1429025673.7346.37.camel-XN9IlZ5yJG9HTL0Zs8A6p/gx64E7kk8eUsxypvmhUTTZJqsBc5GL+g@public.gmane.org> 2015-04-14 15:44 ` Honggang LI 2015-04-14 15:44 ` Honggang LI [not found] ` <20150414154422.GB21856-9l7K0WC0B0wP68cbUhXDDlaTQe2KTcn/@public.gmane.org> 2015-04-14 15:49 ` Nicolas Dichtel 2015-04-14 15:49 ` Nicolas Dichtel [not found] ` <552D3723.9050706-pdR9zngts4EAvxtiuMwx3w@public.gmane.org> 2015-04-14 15:53 ` Honggang LI 2015-04-14 15:53 ` Honggang LI [not found] ` <20150414155307.GD21856-9l7K0WC0B0wP68cbUhXDDlaTQe2KTcn/@public.gmane.org> 2015-04-14 16:14 ` Eric Dumazet 2015-04-14 16:14 ` Eric Dumazet 2015-04-14 16:01 ` Yann Droneaud 2015-04-14 16:01 ` Yann Droneaud [not found] ` <1429027293.4333.5.camel-RlY5vtjFyJ3QT0dZR+AlfA@public.gmane.org> 2015-04-14 16:44 ` Nicolas Dichtel 2015-04-14 16:44 ` Nicolas Dichtel 2015-04-14 16:30 ` Erez Shitrit [not found] ` <CAAk-MO-O9sjHQvDfCEuzJJPvUMXJTuRaCzrCkB0xc1DUfK8Aew-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2015-04-14 16:46 ` Nicolas Dichtel 2015-04-14 16:46 ` Nicolas Dichtel 2015-04-14 20:41 ` Jason Gunthorpe 2015-04-14 20:41 ` Jason Gunthorpe [not found] ` <20150414204133.GJ7682-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org> 2015-04-15 5:24 ` Or Gerlitz 2015-04-15 5:24 ` Or Gerlitz 2015-04-15 5:24 ` Or Gerlitz 2015-04-15 6:17 ` Erez Shitrit 2015-04-15 6:17 ` Erez Shitrit [not found] ` <552E026A.4020200-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org> 2015-04-15 16:06 ` Jason Gunthorpe 2015-04-15 16:06 ` Jason Gunthorpe [not found] ` <20150415160623.GA4653-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org> 2015-04-16 11:27 ` Erez Shitrit 2015-04-16 11:27 ` Erez Shitrit 2015-04-15 5:16 ` Honggang LI 2015-04-15 5:16 ` Honggang LI [not found] ` <20150415051640.GB4881-9l7K0WC0B0wP68cbUhXDDlaTQe2KTcn/@public.gmane.org> 2015-04-15 6:57 ` Honggang LI 2015-04-15 6:57 ` Honggang LI 2015-04-14 15:20 ` [PATCH linux-next 2/4] ipvlan: fix possible NULL pointer dereference in ipvlan_get_iflink Honggang Li [not found] ` <1429024817-21561-1-git-send-email-honli-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2015-04-14 15:20 ` [PATCH linux-next 3/4] macvlan: fix possible NULL pointer dereference in macvlan_dev_get_iflink Honggang Li 2015-04-14 15:20 ` Honggang Li 2015-04-14 15:26 ` Patrick McHardy 2015-04-14 15:32 ` Honggang LI 2015-04-14 15:35 ` Patrick McHardy 2015-04-14 17:47 ` David Miller 2015-04-14 15:35 ` Nicolas Dichtel [not found] ` <552D33B0.6040808-pdR9zngts4EAvxtiuMwx3w@public.gmane.org> 2015-04-14 15:37 ` Andrew Lunn 2015-04-14 15:37 ` Andrew Lunn 2015-04-14 15:46 ` Honggang LI 2015-04-14 16:26 ` [PATCH linux-next v2] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink Honggang Li 2015-04-14 16:26 ` Honggang Li [not found] ` <1429028811-29888-1-git-send-email-honli-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2015-04-14 16:26 ` [PATCH] " Honggang Li 2015-04-14 16:26 ` Honggang Li 2015-04-14 15:20 ` [PATCH linux-next 4/4] net/dsa: fix possible NULL pointer dereference in dsa_slave_get_iflink Honggang Li [not found] ` <1429024817-21561-5-git-send-email-honli-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2015-04-14 15:55 ` Guenter Roeck 2015-04-14 15:55 ` Guenter Roeck
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.