On Saturday, July 18, 2015 at 8:46:21 PM UTC-4, Jeffrey Walton wrote:
>
> Hi Everyone,
>
> Analysis under UBsan revealed undefined behavior, even when 
> CRYPTOPP_ALLOW_UNALIGNED_DATA_ACCESS was in effect. There were two 
> problems. First, assumeAligned was not always correct. Then, IsAligned<T> 
> returns a doctored value of 1 when CRYPTOPP_ALLOW_UNALIGNED_DATA_ACCESS  
> was defined.
>
> The patch below clears the UB. The commented code shown was the original 
> code path. It was modified for testing. It showed up as a change due to a 
> local commit.
>
> Any comments or objections?
>
> **********
>

This was committed at:

 * 
https://github.com/weidai11/cryptopp/commit/e7eee759bbb7b40bd5a52edf220a5fabad77a06c
 * (SourceForge is currently down, no concurrent check-in)

-- 
-- 
You received this message because you are subscribed to the "Crypto++ Users" 
Google Group.
To unsubscribe, send an email to [email protected].
More information about Crypto++ and this group is available at 
http://www.cryptopp.com.
--- 
You received this message because you are subscribed to the Google Groups 
"Crypto++ Users" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to