Support for OpenSSL up to 3.1.x

[fferino-fungible]

No description provided.

[TrinityTonic]

This would actually be quite nice to have. However we tried this with your proposed change and openssl v3 and compilation went successful. However later on when we wantedto use the library, there were some linker errors, that some symbols had been defined multiple times. Honestly I doubt that this would be related to your updates, however I'm not sure. Check the jenkings output where we wanted to link the oss.tpm to one of our apps and use it for testing.

+ ./cx-build build
[golang-lib: build => web.licensemanager-1.5.0-feature.floatingServer.8]
go build -mod=vendor ./...
# lo-stash.de.bosch.com/icx/web.licensemanager.git/cmd/licensemanager
/snap/go/current/pkg/tool/linux_amd64/link: running gcc failed: exit status 1
/usr/bin/ld: /home/jenkins/agent/workspace/semanager_feature_floatingServer@2/deps/ms-tpm-20-ref/amd64/lib/libplatform.a(libplatform_a-PlatformData.o):(.bss+0x8): multiple definition of `s_realTimePrevious'; /tmp/go-link-294997129/000048.o:(.bss+0x8): first defined here
/usr/bin/ld: /home/jenkins/agent/workspace/semanager_feature_floatingServer@2/deps/ms-tpm-20-ref/amd64/lib/libplatform.a(libplatform_a-PlatformData.o):(.bss+0x10): multiple definition of `s_lastSystemTime'; /tmp/go-link-294997129/000048.o:(.bss+0x18): first defined here
/usr/bin/ld: /home/jenkins/agent/workspace/semanager_feature_floatingServer@2/deps/ms-tpm-20-ref/amd64/lib/libplatform.a(libplatform_a-PlatformData.o):(.bss+0x18): multiple definition of `s_lastReportedTime'; /tmp/go-link-294997129/000048.o:(.bss+0x20): first defined here
/usr/bin/ld: /home/jenkins/agent/workspace/semanager_feature_floatingServer@2/deps/ms-tpm-20-ref/amd64/lib/libplatform.a(libplatform_a-PlatformData.o):(.bss+0x20): multiple definition of `s_tpmTime'; /tmp/go-link-294997129/000048.o:(.bss+0x10): first defined here
/usr/bin/ld: /home/jenkins/agent/workspace/semanager_feature_floatingServer@2/deps/ms-tpm-20-ref/amd64/lib/libplatform.a(libplatform_a-PlatformData.o):(.bss+0x28): multiple definition of `s_timerReset'; /tmp/go-link-294997129/000048.o:(.bss+0x4): first defined here
/usr/bin/ld: /home/jenkins/agent/workspace/semanager_feature_floatingServer@2/deps/ms-tpm-20-ref/amd64/lib/libplatform.a(libplatform_a-PlatformData.o):(.bss+0x38): multiple definition of `s_adjustRate'; /tmp/go-link-294997129/000048.o:(.bss+0x0): first defined here
/usr/bin/ld: /home/jenkins/agent/workspace/semanager_feature_floatingServer@2/deps/ms-tpm-20-ref/amd64/lib/libplatform.a(libplatform_a-PlatformData.o):(.bss+0x40): multiple definition of `s_NV'; /tmp/go-link-294997129/000050.o:(.bss+0x0): first defined here
collect2: error: ld returned 1 exit status

[fferino-fungible]

@TrinityTonic it's difficult to be 100% sure what is going on with your build.

However, if you look for the variables that are are reported as multiply defined, they are in the header file with an EXTERN macro :

ms-tpm-20-ref/TPMCmd/Platform/include/PlatformData.h

Line 56 in f74c0d9

EXTERN clock64_t s_realTimePrevious;

.

And this EXTERN macro maps either to nothing or to the "extern" C keyword depending on whether the macro PLATFORM_DATA_C is defined.

ms-tpm-20-ref/TPMCmd/Platform/include/PlatformData.h

Line 41 in f74c0d9

#ifdef _PLATFORM_DATA_C_

So depending on how you build the TPM library, you will either have one, many or no definitions of these variables in your build of the TPM library. In your case, you seem to have either multiple definitions or at least one with another definition provided with the other components in your link so PLATFORM_DATA_C was defined in your build of the TPM library.

I would try first to rebuild the library with PLATFORM_DATA_C undefined.

It will either fix the problem (the variables are provided by another component of your build) or you will get an "undefined" or "missing definition" link error. If the latter, then your toolchain treats uninitialized global variables as common symbols (cf. the "Mac OS X build" of the README.md) https://github.com/microsoft/ms-tpm-20-ref/blob/main/README.md#mac-os-x-build and you will need to change your compiler/linker flags.

I hope this helps.

[wxleong]

Any plan to accept this PR?

[bradlitterell]

Our plan is to update this repo with the next public version of the code which is currently being worked on which has a similar change.

[wxleong]

Any estimated timeframe for this public release?