Commit 3a4691a8 authored by Steven Fackler's avatar Steven Fackler
Browse files

Merge pull request #149 from gkoz/hashers_double_fin 

Avoid duplicate calls to *_Final when dropping Hasher and HMAC
parents 8b47daae 4bd7ed8a

src/crypto/hash.rs

+1 −1
Original line number Diff line number Diff line
@@ -153,10 +153,10 @@ impl Hasher { 
        unsafe {

            let mut len = 0;

            let r = ffi::EVP_DigestFinal_ex(self.ctx, res.as_mut_ptr(), &mut len);

            self.state = Finalized;

            assert_eq!(len as usize, md_len);

            assert_eq!(r, 1);

        }

        self.state = Finalized;

        res

    }

src/crypto/hmac.rs

+1 −1
Original line number Diff line number Diff line
@@ -136,10 +136,10 @@ impl HMAC { 
        unsafe {

            let mut len = 0;

            let r = ffi::HMAC_Final(&mut self.ctx, res.as_mut_ptr(), &mut len);

            self.state = Finalized;

            assert_eq!(len as usize, md_len);

            assert_eq!(r, 1);

        }

        self.state = Finalized;

        res

    }