From 8acf77aede21ca6d82415639557306babe3c71bf Mon Sep 17 00:00:00 2001 From: Benjamin Barenblat Date: Sat, 28 Apr 2018 16:08:54 -0700 Subject: Abstract out common body between hashing and HMACing --- src/Data/Digest/Internal.hs | 48 +++++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 46 insertions(+), 2 deletions(-) (limited to 'src/Data/Digest/Internal.hs') diff --git a/src/Data/Digest/Internal.hs b/src/Data/Digest/Internal.hs index 859f765..02b879c 100644 --- a/src/Data/Digest/Internal.hs +++ b/src/Data/Digest/Internal.hs @@ -12,16 +12,26 @@ -- License for the specific language governing permissions and limitations under -- the License. -module Data.Digest.Internal where +module Data.Digest.Internal + ( Algorithm(..) + , Digest(..) + , initUpdateFinalize + ) where import Data.Bits (Bits((.&.)), shiftR) import Data.ByteString (ByteString) import qualified Data.ByteString as ByteString +import qualified Data.ByteString.Unsafe as ByteString +import qualified Data.ByteString.Lazy as ByteString.Lazy import Data.Char (intToDigit) import Data.Word (Word8) -import Foreign (Ptr) +import Foreign (ForeignPtr, Storable(peek), Ptr, alloca, allocaArray, withForeignPtr) +import Foreign.C.Types import Internal.Base (EVPMD) +import Internal.Digest (evpMaxMDSize) + +type LazyByteString = ByteString.Lazy.ByteString -- | A cryptographic hash function. newtype Algorithm = Algorithm (Ptr EVPMD) @@ -36,3 +46,37 @@ instance Show Digest where showHexPadded b xs = hexit (b `shiftR` 4 .&. 0x0f) : hexit (b .&. 0x0f) : xs hexit = intToDigit . fromIntegral :: Word8 -> Char + +-- | Encapsulates a common pattern of operation between hashing and HMAC +-- computation. Both of these operations require an allocated context local to +-- the operation. The context gets initialized once, updated repeatedly, and +-- then finalized. Finally, we read the result out of a buffer produced by the +-- finalizer. +-- +-- The updater must not mutate any argument other than the context. +-- +-- If all arguments are safe to use under 'unsafeLocalState', this whole +-- function is safe to use under 'unsafeLocalState'. +initUpdateFinalize :: + IO (ForeignPtr ctx) + -> (Ptr ctx -> IO ()) + -> (Ptr ctx -> Ptr CChar -> CULong -> IO ()) + -> (Ptr ctx -> Ptr CChar -> Ptr CUInt -> IO ()) + -> LazyByteString + -> IO ByteString +initUpdateFinalize mallocCtx initialize update finalize bytes = do + ctxFP <- mallocCtx + withForeignPtr ctxFP $ \ctx -> do + initialize ctx + mapM_ (updateBytes ctx) (ByteString.Lazy.toChunks bytes) + allocaArray evpMaxMDSize $ \rOut -> + alloca $ \pOutSize -> do + finalize ctx rOut pOutSize + outSize <- fromIntegral <$> peek pOutSize + ByteString.packCStringLen (rOut, outSize) + where + updateBytes ctx chunk = + -- The updater won't mutate its arguments, so the sharing inherent in + -- 'ByteString.unsafeUseAsCStringLen' is fine. + ByteString.unsafeUseAsCStringLen chunk $ \(buf, len) -> + update ctx buf (fromIntegral len) -- cgit v1.2.3