RE: [staging-next V3] ozwpan: fix memcmp() test in oz_set_active_pd()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



________________________________________
From: Tomas Winkler [tomas.winkler@xxxxxxxxx]
Sent: 19 August 2012 17:36
To: gregkh@xxxxxxxxxxxxxxxxxxx
Cc: alan@xxxxxxxxxxxxxxx; devel@xxxxxxxxxxxxxxxxxxxxxx; kernel-janitors@xxxxxxxxxxxxxxx; Tomas Winkler; Dan Carpenter; Rupesh Gujare
Subject: [staging-next V3] ozwpan: fix memcmp() test in oz_set_active_pd()

"addr" is a pointer so it's either 4 or 8 bytes, but actually we want
to compare 6 bytes (ETH_ALEN).

As network stack already provides helper function
is_zero_ether_addr() we use that instead of memcmp

Signed-off-by: Tomas Winkler <tomas.winkler@xxxxxxxxx>
Cc: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
Cc: Rupesh Gujare <rgujare@xxxxxxxxxxxxxxx>
---
V1: by Dan Carpenter <dan.carpenter@xxxxxxxxxx>
V2: replace memcmp with is_zero_ether_addr
V3: include <linux/etherdevice.h>

Acked-by: Rupesh Gujare<rgujare@xxxxxxxxxxxxxxx> 


--
Regards,
Rupesh Gujare
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel


[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux