Re: [PATCH] mac80211: add explicit padding in structieee80211_tx_info

From: Johannes Berg
Date: Wed Nov 12 2008 - 12:08:57 EST


On Wed, 2008-11-12 at 10:05 -0500, John W. Linville wrote:
> Otherwise, the BUILD_BUG_ON calls in ieee80211_tx_info_clear_status can
> fail on some architectures.
>
> Signed-off-by: John W. Linville <linville@xxxxxxxxxxxxx>
> ---
> Why do we have those BUILD_BUG_ON calls anyway? Hmmm...

Because we actually use those offsets in places to clear things.

> include/net/mac80211.h | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/include/net/mac80211.h b/include/net/mac80211.h
> index feb52cc..38302d6 100644
> --- a/include/net/mac80211.h
> +++ b/include/net/mac80211.h
> @@ -333,6 +333,7 @@ struct ieee80211_tx_info {
> u8 antenna_sel_tx;
>
> /* 2 byte hole */
> + u8 pad[2];
>
> union {
> struct {

Attachment: signature.asc
Description: This is a digitally signed message part