This is an archive of the discontinued LLVM Phabricator instance.

Add interceptors for the rmd160(3) from NetBSD
ClosedPublic

Authored by krytarowski on Nov 27 2018, 10:28 PM.

Details

Summary

RMD160Init, RMD160Update, RMD160Final, RMD160Transform, RMD160End,
RMD160File, RMD160Data - calculates the ``RIPEMD-160'' message digest.

Add a dedicated test for this API.

Diff Detail

Event Timeline

krytarowski created this revision.Nov 27 2018, 10:28 PM
vitalybuka requested changes to this revision.Dec 5 2018, 1:41 AM
vitalybuka added inline comments.
lib/sanitizer_common/sanitizer_common_interceptors.inc
7378

should write context

7385

should read context

7392

should write something, maybe both

This revision now requires changes to proceed.Dec 5 2018, 1:41 AM
  • fixes
  • align style
vitalybuka added inline comments.Dec 7 2018, 4:02 PM
lib/sanitizer_common/sanitizer_common_interceptors.inc
7376

Is there any issues with COMMON_INTERCEPTOR_READ_RANGE when len is 0?

7392

so why this does not write?

krytarowski marked 2 inline comments as done.Dec 7 2018, 4:11 PM
krytarowski added inline comments.
lib/sanitizer_common/sanitizer_common_interceptors.inc
7376

at least it's waste operation and doesn't make sense

7392

Forgotten, to be fixed.

  • add missing write
krytarowski marked 4 inline comments as done.Dec 7 2018, 4:13 PM
This revision is now accepted and ready to land.Dec 7 2018, 4:16 PM
This revision was automatically updated to reflect the committed changes.