[v2] batman-adv: use the name of the array as pointer

Message ID 1393420412-3931-1-git-send-email-antonio@meshcoding.com (mailing list archive)
State Accepted, archived
Headers

Commit Message

Antonio Quartulli Feb. 26, 2014, 1:13 p.m. UTC
  Introduced by fb4756fe5f6e58ca2ab208ca634ae4c8dd6e9499
("batman-adv: use ether_addr_copy instead of memcpy in multicast code")

Signed-off-by: Antonio Quartulli <antonio@meshcoding.com>
---

change from v1:
- added Signed-off-by clause

 multicast.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Marek Lindner March 11, 2014, 9:55 a.m. UTC | #1
On Wednesday 26 February 2014 14:13:32 Antonio Quartulli wrote:
> Introduced by fb4756fe5f6e58ca2ab208ca634ae4c8dd6e9499
> ("batman-adv: use ether_addr_copy instead of memcpy in multicast code")
> 
> Signed-off-by: Antonio Quartulli <antonio@meshcoding.com>
> ---
> 
> change from v1:
> - added Signed-off-by clause
> 
>  multicast.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied in revision f5129b2.

Thanks,
Marek
  

Patch

diff --git a/multicast.c b/multicast.c
index e69143b..998b429 100644
--- a/multicast.c
+++ b/multicast.c
@@ -45,7 +45,7 @@  static int batadv_mcast_mla_softif_get(struct net_device *dev,
 			break;
 		}
 
-		ether_addr_copy(&new->addr, &mc_list_entry->addr);
+		ether_addr_copy(new->addr, mc_list_entry->addr);
 		hlist_add_head(&new->list, mcast_list);
 		ret++;
 	}