netdev - Re: octeon dev_addrs list usage

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: <4B856B87.9030507@caviumnetworks.com>
Date:	Wed, 24 Feb 2010 10:10:15 -0800
From:	David Daney <ddaney@...iumnetworks.com>
To:	Jiri Pirko <jpirko@...hat.com>
CC:	davem@...emloft.net, netdev@...r.kernel.org
Subject: Re: octeon dev_addrs list usage

On 02/23/2010 12:41 PM, Jiri Pirko wrote:
> Hello David.
>
> Lookin at octeon driver, drivers/net/octeon/octeon_mgmt.c,
> octeon_mgmt_set_rx_filtering function and following code:
>
> 503         if (cam_mode == 1) {
> 504                 /* Add primary address. */
> 505                 octeon_mgmt_cam_state_add(&cam_state, netdev->dev_addr);
> 506                 list_for_each(pos,&netdev->dev_addrs.list) {
> 507                         struct netdev_hw_addr *hw_addr;
> 508                         hw_addr = list_entry(pos, struct netdev_hw_addr, list);
> 509                         octeon_mgmt_cam_state_add(&cam_state, hw_addr->addr);
> 510                         list = list->next;
> 511                 }
> 512         }
>
> besides netdev->dev_addr is first entry in netdev->dev_addrs.list and

Ok.

> "list = list->next;" makes no sense here (I posted a patch correcting
> these earlier),

Right, that does look wrong.


> are you sure you are traversing the right list? This list is
> currently not filled in anywhere (besides ixgbe driver). Don't you want
> to traverse through unicast_list instead? Maybe I'm missing something...
>

You are likely right. I will revisit this code, but it could be a couple 
of days.

Thanks for looking at it,
David Daney


--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html