[PATCH] hostap: use offsetof() instead of own equivalent
authorPavel Roskin <proski@gnu.org>
Mon, 19 Feb 2007 01:44:06 +0000 (20:44 -0500)
committerJeff Garzik <jeff@garzik.org>
Sat, 28 Apr 2007 15:00:55 +0000 (11:00 -0400)
The original macros result in gcc 4.2 warning about "cast from pointer
to integer of different size" on 64-bit systems.

Use of offsetof() on fields in substructures is widespread throughout
the kernel code and should work whether offsetof() is defined using
__compiler_offsetof() or a cast.

Signed-off-by: Pavel Roskin <proski@gnu.org>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/hostap/hostap_common.h

index 01624005d808f541ae30f762ab5e4dad1785ff8e..b31e6a05f23cc5a97a5b35fa448a12896eed1ed9 100644 (file)
@@ -368,9 +368,9 @@ enum {
 
 #define PRISM2_HOSTAPD_MAX_BUF_SIZE 1024
 #define PRISM2_HOSTAPD_RID_HDR_LEN \
-((int) (&((struct prism2_hostapd_param *) 0)->u.rid.data))
+offsetof(struct prism2_hostapd_param, u.rid.data)
 #define PRISM2_HOSTAPD_GENERIC_ELEMENT_HDR_LEN \
-((int) (&((struct prism2_hostapd_param *) 0)->u.generic_elem.data))
+offsetof(struct prism2_hostapd_param, u.generic_elem.data)
 
 /* Maximum length for algorithm names (-1 for nul termination) used in ioctl()
  */