Skip to content

PS-10991-8.0 submodule(extra/libkmip): update to libkmip@7e809c0 (master, 2026-04-02)#5908

Open
lukin-oleksiy wants to merge 1 commit intopercona:8.0from
lukin-oleksiy:PS-10991-update-kmip-submodule
Open

PS-10991-8.0 submodule(extra/libkmip): update to libkmip@7e809c0 (master, 2026-04-02)#5908
lukin-oleksiy wants to merge 1 commit intopercona:8.0from
lukin-oleksiy:PS-10991-update-kmip-submodule

Conversation

@lukin-oleksiy
Copy link
Copy Markdown
Contributor

This is commit to 8.0 branch only, in higher version we'll use new kmipclient library

https://perconadev.atlassian.net/browse/PS-10991

Previous: d855fbf (0.3.2-4-gd855fbf)
Updated to Percona-Lab/libkmip origin/master to pick up:

  • Fix revoke/SecretData decode leaks and add ASAN test target (PS-10949)

… 2026-04-02)

https://perconadev.atlassian.net/browse/PS-10991

Previous: d855fbf (0.3.2-4-gd855fbf)
Updated to Percona-Lab/libkmip origin/master to pick up:
- Fix revoke/SecretData decode leaks and add ASAN test target (PS-10949)
@lukin-oleksiy lukin-oleksiy requested a review from inikep April 2, 2026 14:53
Copy link
Copy Markdown
Collaborator

@inikep inikep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Azure Pipelines reports compilation issues with clang:

[  6%] Building C object extra/libkmip/libkmip/src/CMakeFiles/kmip.dir/kmip_bio.c.o
../../../../../extra/libkmip/libkmip/src/kmip_bio.c:292:12: warning: assigning to 'uint8 *' (aka 'unsigned char *') from 'char *' converts between pointers to integer types where one is of the unique plain 'char' type and the other is not [-Wpointer-sign]
  292 |   bs.value = key;
      |            ^ ~~~
../../../../../extra/libkmip/libkmip/src/kmip_bio.c:2696:12: warning: assigning to 'uint8 *' (aka 'unsigned char *') from 'char *' converts between pointers to integer types where one is of the unique plain 'char' type and the other is not [-Wpointer-sign]
 2696 |   bs.value = secret;
      |            ^ ~~~~~~

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants