Skip to content

[PATCH 0/4] uadk_provider: several improvements and fixes#260

Merged
lin755 merged 4 commits into
Linaro:masterfrom
DRBin3:0623
Jun 24, 2026
Merged

[PATCH 0/4] uadk_provider: several improvements and fixes#260
lin755 merged 4 commits into
Linaro:masterfrom
DRBin3:0623

Conversation

@DRBin3

@DRBin3 DRBin3 commented Jun 23, 2026

Copy link
Copy Markdown

From: Weili Qian <qianweili@huawei.com>

*** BLURB HERE ***

Weili Qian (3):
uadk_provider: set rsa sign hash flag
uadk_provider: add outsize check in uadk_asym_cipher_rsa_encrypt
uadk_provider: remove soft field to match default provider structure
size

lizhi (1):
uadk_provider: fix return UADK_P_FAIL when alloc sess fails in
ec_alloc_sess

src/uadk_prov_ec_kmgmt.c | 2 +-
src/uadk_prov_rsa.c | 9 ++++--
src/uadk_prov_rsa.h | 1 -
src/uadk_prov_rsa_enc.c | 35 ++++++++++++----------
src/uadk_prov_rsa_kmgmt.c | 6 ----
src/uadk_prov_rsa_sign.c | 61 ++++++++++++++++++++++++++++-----------
6 files changed, 72 insertions(+), 42 deletions(-)

--
2.43.0

Weili Qian and others added 4 commits June 23, 2026 15:40
Set rsa signature hash flag, this flag is cleared by their
Init function, and set again by their Final function.

Signed-off-by: Weili Qian <qianweili@huawei.com>
Add explicit outsize check to ensure the output buffer is large enough
before proceeding with RSA encryption operation.

Signed-off-by: Weili Qian <qianweili@huawei.com>
…size

Remove the soft field from PROV_RSA_SIG_CTX and PROV_RSA_ASYM_CTX to ensure
structure size consistency with the default provider, which is required for
proper context duplication via dupctx callback.

UADK-specific initialization is performed before session allocation
to compensate for the removed field.

Signed-off-by: Weili Qian <qianweili@huawei.com>
…oc_sess

Return UADK_P_FAIL when sess allocation fails in ec_alloc_sess.

Signed-off-by: lizhi <lizhi206@huawei.com>
@qwl731

qwl731 commented Jun 23, 2026

Copy link
Copy Markdown
Collaborator

/lgtm

@Liulongfang Liulongfang left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good to me

@lin755 lin755 merged commit a8c9352 into Linaro:master Jun 24, 2026
2 checks passed
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.

4 participants