Re: [PATCH] crypto: chacha20 - Fix chacha20_block() keystream alignment (again)

2018-09-12 Thread Yann Droneaud
s introduces a alignment issue: tmp is not aligned for __u32, but is dereferenced as such later. > d = >state[4]; > for (i=0; i < 8; i++) > *d++ ^= *s++; > spin_unlock_irqrestore(>lock, flags); > } > Regards. -- Yann Droneaud OPTEYA

Re: random: ensure use of aligned buffers with ChaCha20

2018-08-09 Thread Yann Droneaud
t; spin_lock_irqsave(>lock, flags); > - s = [used / sizeof(__u32)]; > + s = (__u32 *) [used]; tmp is __8* while s is __u32*, that sounds like an alignment issue ... > d = >state[4]; > for (i=0; i < 8; i++) > *d++ ^= *s++; ... when s is dereferenced > spin_unlock_irqrestore(>lock, flags); > } > Regards. -- Yann Droneaud OPTEYA