One thing that makes me cringe is when people randomly invent their own cryptographic protocols. There's a Google Tech Talk by Nate Lawson where he explains some surprising approaches to attacking a cryptographic algorithm. It illustrates why rolling your own is probably a bad idea ;-)
Perhaps the most NIH cryptographic protocol I've seen is digitally signing as well as encrypting a message, in order to store tamper resistant data without revealing its contents. This is often done for storing sensitive data in cookies.
Obviously such a protocol can be built using HMACs and ciphers, but high level tools are already available, ones that have already been designed and analyzed by people who actually know what they're doing: authenticated encryption modes of operation.
WTF is a cipher mode?
Block ciphers are the sort of like hash functions, they take a block of data and scramble the block.
Simply encrypting your data blocks one by one is not a good way of securing it though. Wikipedia has a striking example:
Even though every pixel is encrypted, the data as a whole still reveals a lot.
Suffice it to say that blocks of operation are a wrapper that takes a low level scrambling function, the block cipher, and provide a less error prone tool, one that is more difficult to misuse.
On the CPAN
Crypt::CBC and Crypt::Ctr are implementations of some of the more classic cipher modes. But this post is ranting about people not using authenticated modes.
Crypt::GCM and Crypt::EAX implement two different AEAD modes of operation using any block cipher.
These are carefully designed and analyzed algorithms, and the CPAN implementations make use of the tests from the articles describing the algorithms, so it sure beats rolling your own.
Secondly, Crypt::Util provides a convenience layer that builds on these tools (and many others), so perhaps Crypt::Util already handles what you want.
To tamper protect a simple data structure you can do something like this:
my $cu = Crypt::Util->new( key => ... ); my $ciphertext = $cu->tamper_proof_data( data = { ... }, encrypt => 1 );
Crypt::Util will use Storable to encode the data into a string, and then use an authenticated encryption mode to produce the ciphertext.
To decrypt, simply do:
my $data = $c->thaw_tamper_proof( string => $ciphertext );
Crypt::Util will decrypt and validate the ciphertext, and only after it's sure that the data is trusted it'll start unpacking the data, and if appropriate using Storable to deserialize the message. All allocations based on untrusted data are limited to 64KiB.
Don't sue me
I'm not saying that the CPAN code is guaranteed to be safe. I'm saying this is a better idea than rolling your own. If your application is sensitive you have no excuse not to open up the code and audit it.
No comments:
Post a Comment