Skip to content

Complete the zeroize implementation on the Precomputed - #708

Draft
baloo wants to merge 1 commit into
RustCrypto:masterfrom
baloo:baloo/push-soqyyxrrszkx
Draft

Complete the zeroize implementation on the Precomputed#708
baloo wants to merge 1 commit into
RustCrypto:masterfrom
baloo:baloo/push-soqyyxrrszkx

Conversation

@baloo

@baloo baloo commented Sep 2, 2026

Copy link
Copy Markdown
Member

@tarcieri

tarcieri commented Sep 4, 2026

Copy link
Copy Markdown
Member

Perhaps PrecomputedValues could be changed to store MontyParams<BoxedUint> for p_params and q_params which would allow the Zeroize impl on that to be used directly and avoid any Arc-related issues from BoxedMontyParams.

I'm uncertain if everything p_params/q_params require can be done via the MontyParams API, but if not we could potentially add something like BoxedMontyParams::into_inner which can in turn call Arc::into_inner.

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