Skip to content

Conversation

@hoyhoy
Copy link

@hoyhoy hoyhoy commented Jan 8, 2026

Fix for gcc 15.2 compiler error.

azure-security-attestation/src/private/crypto/openssl/opensslcrypto.cpp:11:
/azure4e924e642f1fd/b/src/sdk/attestation/azure-security-attestation/src/private/crypto/openssl/../inc/crypto.hpp:42:23: error: ‘uint8_t’ was not declared in this scope
   42 |           std::vector<uint8_t> const& payload,
      |                       ^~~~~~~
/azure4e924e642f1fd/b/src/sdk/attestation/azure-security-attestation/src/private/crypto/openssl/../inc/crypto.hpp:9:1: note: ‘uint8_t’ is defined in header ‘<cstdint>’; this is probably fixable by adding ‘#include <cstdint>’
    8 | #include <vector>
  +++ |+#include <cstdint>
    9 | 
/azure4e924e642f1fd/b/src/sdk/attestation/azure-security-attestation/src/private/crypto/openssl/../inc/crypto.hpp:42:30: error: template argument 1 is invalid
   42 |           std::vector<uint8_t> const& payload,
      |                              ^
/azure4e924e642f1fd/b/src/sdk/attestation/azure-security-attestation/src/private/crypto/openssl/../inc/crypto.hpp:42:30: error: template argument 2 is invalid
/azure4e924e642f1fd/b/src/sdk/attestation/azure-security-attestation/src/private/crypto/openssl/../inc/crypto.hpp:43:23: error: ‘uint8_t’ was not declared in this scope
  • C++ Guidelines
  • Doxygen docs
  • Unit tests
  • No unwanted commits/changes
  • Descriptive title/description
    • PR is single purpose
    • Related issue listed
  • Comments in source
  • No typos
  • Update changelog
  • Not work-in-progress
  • External references or docs updated
  • Self review of PR done
  • Any breaking changes?

@github-actions github-actions bot added Attestation Community Contribution Community members are working on the issue customer-reported Issues that are reported by GitHub users external to the Azure organization. labels Jan 8, 2026
@github-actions
Copy link

github-actions bot commented Jan 8, 2026

Thank you for your contribution @hoyhoy! We will review the pull request and get back to you soon.

@hoyhoy
Copy link
Author

hoyhoy commented Jan 8, 2026

@microsoft-github-policy-service agree

@hoyhoy
Copy link
Author

hoyhoy commented Jan 14, 2026

Thanks @LarryOsterman...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Attestation Community Contribution Community members are working on the issue customer-reported Issues that are reported by GitHub users external to the Azure organization.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants