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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sat, 08 Mar 2014 19:01:52 -0500 (EST) From: David Miller <davem@...emloft.net> To: saproj@...il.com Cc: behanw@...verseincode.com, linville@...driver.com, johannes@...solutions.net, linux-wireless@...r.kernel.org, netdev@...r.kernel.org, linux-kernel@...r.kernel.org, dwmw2@...radead.org, pageexec@...email.hu, dl9pf@....de, viniciustinti@...il.com, charlebm@...il.com Subject: Re: [PATCH v2] mac80211: LLVMLinux: Remove VLAIS usage from mac80211 From: Sergei Antonov <saproj@...il.com> Date: Sat, 8 Mar 2014 21:29:57 +0100 > On 8 March 2014 02:26, <behanw@...verseincode.com> wrote: >> diff --git a/net/mac80211/aes_ccm.c b/net/mac80211/aes_ccm.c >> index 7c7df47..3317578 100644 >> --- a/net/mac80211/aes_ccm.c >> +++ b/net/mac80211/aes_ccm.c >> @@ -23,12 +23,14 @@ void ieee80211_aes_ccm_encrypt(struct crypto_aead *tfm, u8 *b_0, u8 *aad, >> u8 *data, size_t data_len, u8 *mic) >> { >> struct scatterlist assoc, pt, ct[2]; >> - struct { >> - struct aead_request req; >> - u8 priv[crypto_aead_reqsize(tfm)]; >> - } aead_req; >> >> - memset(&aead_req, 0, sizeof(aead_req)); >> + char aead_req_data[sizeof(struct aead_request) + >> + crypto_aead_reqsize(tfm) + >> + CRYPTO_MINALIGN] CRYPTO_MINALIGN_ATTR; >> + >> + struct aead_request *aead_req = (void *) aead_req_data; > > Bad trick with regards to alignment. > The alignment requirement for struct aead_request is stronger than > what an array of chars may have. Indeed, this will crash on platforms such as sparc64 that trap on unaligned accesses. You have to tell the compiler the alignment using proper types. -- 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
Powered by blists - more mailing lists