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, Mar 10, 3:01 PM
Unknown Object (File)
Sat, Mar 8, 3:00 PM
Unknown Object (File)
Thu, Feb 27, 3:15 AM
Unknown Object (File)
Wed, Feb 26, 10:47 AM
Unknown Object (File)
Wed, Feb 26, 12:13 AM
Unknown Object (File)
Feb 10 2025, 5:27 AM
Unknown Object (File)
Feb 7 2025, 7:34 AM
Unknown Object (File)
Feb 7 2025, 6:34 AM
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
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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
499–500

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
499–500

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.