Page MenuHomeFreeBSD

libmd: stop exporting _block symbols
ClosedPublic

Authored by kevans on Mar 9 2022, 3:34 AM.
Tags
None
Referenced Files
F115870026: D34498.diff
Tue, Apr 29, 7:50 PM
Unknown Object (File)
Mon, Apr 28, 8:53 AM
Unknown Object (File)
Sat, Apr 26, 8:10 AM
Unknown Object (File)
Thu, Apr 24, 11:48 AM
Unknown Object (File)
Tue, Apr 1, 7:52 PM
Unknown Object (File)
Mon, Mar 31, 7:55 PM
Unknown Object (File)
Mar 10 2025, 3:01 PM
Unknown Object (File)
Mar 8 2025, 3:00 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.