netdev - RE: [PATCH net-next 2/8] ieee802154: add address struct with proper endiannes and some operations

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <063D6719AE5E284EB5DD2968C1650D6D0F6DDDE4@AcuExch.aculab.com>
Date:	Fri, 14 Mar 2014 16:00:39 +0000
From:	David Laight <David.Laight@...LAB.COM>
To:	'Phoebe Buckheister' <phoebe.buckheister@...m.fraunhofer.de>,
	"netdev@...r.kernel.org" <netdev@...r.kernel.org>
CC:	"linux-zigbee-devel@...ts.sourceforge.net" 
	<linux-zigbee-devel@...ts.sourceforge.net>,
	"davem@...emloft.net" <davem@...emloft.net>
Subject: RE: [PATCH net-next 2/8] ieee802154: add address struct with proper
 endiannes and some operations

From: Phoebe Buckheister
> Add a replacement ieee802154_addr struct with proper endianness on
> fields. Short address fields are stored as __le16 as on the network,
> extended (EUI64) addresses are __le64 as opposed to the u8[8] format
> used previously. This disconnect with the netdev address, which is
> stored as big-endian u8[8], is intentional.
...
> +struct ieee802154_addr {
> +	u8 mode;
> +	__le16 pan_id;
> +	union {
> +		__le16 short_addr;
> +		__le64 extended_addr;
> +	};
> +};

There is a lot of padding in there - especially on 64bit systems.
You didn't make it clear where the above is used, but if it is
passed to userspace I'd add explicit padding fields to ensure
that the alignment is the same on all (sensible) architectures.

	David



--
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