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)
Mon, Jan 27, 5:00 PM
Unknown Object (File)
Sat, Jan 25, 12:46 AM
Unknown Object (File)
Thu, Jan 23, 6:50 PM
Unknown Object (File)
Dec 22 2024, 10:45 AM
Unknown Object (File)
Dec 13 2024, 4:28 AM
Unknown Object (File)
Nov 28 2024, 11:20 PM
Unknown Object (File)
Nov 25 2024, 1:12 AM
Unknown Object (File)
Nov 23 2024, 1:27 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.