netdev - Re: [PATCH net-next] cxgb4/cxgb4vf: Fix mac

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20181119192431.GC25178@mtr-leonro.mtl.com>
Date:   Mon, 19 Nov 2018 21:24:31 +0200
From:   Leon Romanovsky <leon@...nel.org>
To:     Arjun Vynipadath <arjun@...lsio.com>
Cc:     netdev@...r.kernel.org, davem@...emloft.net, nirranjan@...lsio.com,
        indranil@...lsio.com, dt@...lsio.com,
        Casey Leedom <leedom@...lsio.com>,
        Ganesh Goudar <ganeshgr@...lsio.com>
Subject: Re: [PATCH net-next] cxgb4/cxgb4vf: Fix mac_hlist initialization and
 free

On Mon, Nov 19, 2018 at 03:37:34PM +0530, Arjun Vynipadath wrote:
> Null pointer dereference seen when cxgb4vf driver is unloaded
> without bringing up any interfaces, moving mac_hlist initialization
> to driver probe and free the mac_hlist in remove to fix this.
>
> Fixes 'commit 24357e06ba51 ("cxgb4vf: fix memleak in mac_hlist
> initialization")'
>

Please don't break Fixes line (it is needed to various scripts) and no
extra line between Fixes and SOB.

Thanks

> Signed-off-by: Arjun Vynipadath <arjun@...lsio.com>
> Signed-off-by: Casey Leedom <leedom@...lsio.com>
> Signed-off-by: Ganesh Goudar <ganeshgr@...lsio.com>
> ---
>  drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c     | 19 ++++++++++---------
>  drivers/net/ethernet/chelsio/cxgb4vf/cxgb4vf_main.c |  6 +++---
>  2 files changed, 13 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c b/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c
> index 956e708..cdd6f48 100644
> --- a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c
> +++ b/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c
> @@ -2280,8 +2280,6 @@ static int cxgb_up(struct adapter *adap)
>  #if IS_ENABLED(CONFIG_IPV6)
>  	update_clip(adap);
>  #endif
> -	/* Initialize hash mac addr list*/
> -	INIT_LIST_HEAD(&adap->mac_hlist);
>  	return err;
>
>   irq_err:
> @@ -2295,8 +2293,6 @@ static int cxgb_up(struct adapter *adap)
>
>  static void cxgb_down(struct adapter *adapter)
>  {
> -	struct hash_mac_addr *entry, *tmp;
> -
>  	cancel_work_sync(&adapter->tid_release_task);
>  	cancel_work_sync(&adapter->db_full_task);
>  	cancel_work_sync(&adapter->db_drop_task);
> @@ -2306,11 +2302,6 @@ static void cxgb_down(struct adapter *adapter)
>  	t4_sge_stop(adapter);
>  	t4_free_sge_resources(adapter);
>
> -	list_for_each_entry_safe(entry, tmp, &adapter->mac_hlist, list) {
> -		list_del(&entry->list);
> -		kfree(entry);
> -	}
> -
>  	adapter->flags &= ~FULL_INIT_DONE;
>  }
>
> @@ -5629,6 +5620,9 @@ static int init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
>  			     (is_t5(adapter->params.chip) ? STATMODE_V(0) :
>  			      T6_STATMODE_V(0)));
>
> +	/* Initialize hash mac addr list */
> +	INIT_LIST_HEAD(&adapter->mac_hlist);
> +
>  	for_each_port(adapter, i) {
>  		netdev = alloc_etherdev_mq(sizeof(struct port_info),
>  					   MAX_ETH_QSETS);
> @@ -5907,6 +5901,7 @@ static int init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
>  static void remove_one(struct pci_dev *pdev)
>  {
>  	struct adapter *adapter = pci_get_drvdata(pdev);
> +	struct hash_mac_addr *entry, *tmp;
>
>  	if (!adapter) {
>  		pci_release_regions(pdev);
> @@ -5956,6 +5951,12 @@ static void remove_one(struct pci_dev *pdev)
>  		if (adapter->num_uld || adapter->num_ofld_uld)
>  			t4_uld_mem_free(adapter);
>  		free_some_resources(adapter);
> +		list_for_each_entry_safe(entry, tmp, &adapter->mac_hlist,
> +					 list) {
> +			list_del(&entry->list);
> +			kfree(entry);
> +		}
> +
>  #if IS_ENABLED(CONFIG_IPV6)
>  		t4_cleanup_clip_tbl(adapter);
>  #endif
> diff --git a/drivers/net/ethernet/chelsio/cxgb4vf/cxgb4vf_main.c b/drivers/net/ethernet/chelsio/cxgb4vf/cxgb4vf_main.c
> index 8ec503c..8a2ad6b 100644
> --- a/drivers/net/ethernet/chelsio/cxgb4vf/cxgb4vf_main.c
> +++ b/drivers/net/ethernet/chelsio/cxgb4vf/cxgb4vf_main.c
> @@ -723,9 +723,6 @@ static int adapter_up(struct adapter *adapter)
>  		if (adapter->flags & USING_MSIX)
>  			name_msix_vecs(adapter);
>
> -		/* Initialize hash mac addr list*/
> -		INIT_LIST_HEAD(&adapter->mac_hlist);
> -
>  		adapter->flags |= FULL_INIT_DONE;
>  	}
>
> @@ -3038,6 +3035,9 @@ static int cxgb4vf_pci_probe(struct pci_dev *pdev,
>  	if (err)
>  		goto err_unmap_bar;
>
> +	/* Initialize hash mac addr list */
> +	INIT_LIST_HEAD(&adapter->mac_hlist);
> +
>  	/*
>  	 * Allocate our "adapter ports" and stitch everything together.
>  	 */
> --
> 2.9.5
>

Download attachment "signature.asc" of type "application/pgp-signature" (802 bytes)