Skip to content
This repository was archived by the owner on Nov 6, 2020. It is now read-only.

Remove the Keccak C library and use the pure Rust impl #8657

Merged
merged 15 commits into from
May 20, 2018
Merged

Remove the Keccak C library and use the pure Rust impl #8657

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
9c65009
Add license and readme
dvdplm May 15, 2018
df0ebc1
Use pure rust implementation
dvdplm May 18, 2018
eb614b0
Bump version to 0.1.1
dvdplm May 15, 2018
c9adf4e
Don't use C, prefer the pure Rust implementation
dvdplm May 16, 2018
7de620a
Add test for `write_keccak`
dvdplm May 16, 2018
66d36d9
Bump version
dvdplm May 16, 2018
ac9a431
Add benchmarks
dvdplm May 16, 2018
3276b00
Add benchmarks
dvdplm May 18, 2018
000d12f
Add keccak_256, keccak_512, keccak_256_unchecked and keccak_512_unche…
dvdplm May 18, 2018
64f6440
Remove failed git merge attempt from external git repo
dvdplm May 18, 2018
5933f02
whitespace
dvdplm May 18, 2018
4db027a
Mark unsafe function unsafe
dvdplm May 18, 2018
8b0db76
Unsafe calls in unsafe block
dvdplm May 18, 2018
aa19995
Document unsafety invariants
dvdplm May 18, 2018
421c922
Revert unintended changes to Cargo.lock
dvdplm May 19, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 10 additions & 14 deletions util/hash/src/lib.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,22 +39,18 @@ pub fn keccak<T: AsRef<[u8]>>(s: T) -> H256 {
H256(result)
}

pub fn keccak_256_unchecked(out: *mut u8, outlen: usize, input: *const u8, inputlen: usize) {
unsafe {
Keccak::keccak256(
slice::from_raw_parts(input, inputlen),
slice::from_raw_parts_mut(out, outlen)
);
}
pub unsafe fn keccak_256_unchecked(out: *mut u8, outlen: usize, input: *const u8, inputlen: usize) {
Copy link
Contributor

@rphmeier rphmeier May 18, 2018 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's document these and their invariants as well -- general style for the codebase is to provide a little bit of documentation on every public item, but this is especially important for unsafe functions.

Keccak::keccak256(
slice::from_raw_parts(input, inputlen),
slice::from_raw_parts_mut(out, outlen)
);
}

pub fn keccak_512_unchecked(out: *mut u8, outlen: usize, input: *const u8, inputlen: usize) {
unsafe {
Keccak::keccak512(
slice::from_raw_parts(input, inputlen),
slice::from_raw_parts_mut(out, outlen)
);
}
pub unsafe fn keccak_512_unchecked(out: *mut u8, outlen: usize, input: *const u8, inputlen: usize) {
Keccak::keccak512(
slice::from_raw_parts(input, inputlen),
slice::from_raw_parts_mut(out, outlen)
);
}

pub fn keccak_256(input: &[u8], mut output: &mut [u8]) { Keccak::keccak256(input, &mut output); }
Expand Down