Page MenuHomeFreeBSD

libmd: stop exporting _block symbols
ClosedPublic

Authored by kevans on Mar 9 2022, 3:34 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Oct 18, 12:34 AM
Unknown Object (File)
Wed, Oct 16, 5:25 AM
Unknown Object (File)
Wed, Oct 16, 3:09 AM
Unknown Object (File)
Mon, Oct 14, 7:37 PM
Unknown Object (File)
Mon, Oct 14, 12:01 PM
Unknown Object (File)
Mon, Oct 14, 12:01 PM
Unknown Object (File)
Sat, Oct 12, 12:12 AM
Unknown Object (File)
Wed, Oct 9, 5:56 PM
Subscribers

Details

Summary

These are needed across compilation units so we can keep the _libmd_
prefixing bits (though I suspect we're not likely to collide), but we
don't need to be exporting the unprefixed versions of these; it's an
implementation detail.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 44712
Build 41600: arc lint + arc unit

Event Timeline

kevans requested review of this revision.Mar 9 2022, 3:34 AM

These should also be removed from <sha.h>. This change will make D45444 easier as I don't have to stick to the prototype imposed by the old code.

lib/libmd/sha1c.c
513

Do we also still need the _version symbols? They seem to be specific to SSLeay's implementation and are probably not needed anymore.

lib/libmd/sha1c.c
513

I am not aware of any consumers of it. If you do remove it, also remove RMD160_version

So we have no old binaries with the *_block symbols to resolve? I'm skeptical of that notion.

What about the _Transform symbols? These also do not seem to be declared in the header files.

An exp-run has been requested to see if we can safely remove the symbols: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=280784

This revision is now accepted and ready to land.Aug 12 2024, 10:59 PM

Exp-run passed, LGTM.

Note that the exp-run contains a change set to remove the _version symbols, which should also be removed on commit.

This revision was automatically updated to reflect the committed changes.