openbsd/src/lib/libcrypto/bn, branch OPENBSD_7_7A mirror of https://github.com/libressl/openbsd.git
https://git.lua4.win/openbsd/atom?h=OPENBSD_7_72025-03-09T15:22:40+00:00const correct BN_MONT_CTX_copy()2025-03-09T15:22:40+00:00tb2025-03-09T15:22:40+00:00urn:sha1:5631334d720cef2b52503f79b8fc027aa70e1ac8
ok jsing
Convert bn_exp to BN_MONT_CTX_create()2025-02-13T11:15:09+00:00tb2025-02-13T11:15:09+00:00urn:sha1:aa1d25c8cda35c34eff58c5d6568560b729ba8bf
This simplifies the handling of the BN_MONT_CTX passed in and unifies the
exit paths. Also zap some particularly insightful comments by our favorite
captain.
ok jsing
Convert BPSW to BN_MONT_CTX_create()2025-02-13T11:10:01+00:00tb2025-02-13T11:10:01+00:00urn:sha1:66f3a7d5a77ccfa87d46300c9b7cc772d4691dd4
ok jsing
Convert BN_MONT_CTX_set_locked() to BN_MONT_CTX_create()2025-02-13T11:09:51+00:00tb2025-02-13T11:09:51+00:00urn:sha1:a7c9716d89f64dd4a4bae72a3928c9d9fc5d7ef8
ok jsing
bn: add internal BN_MONT_CTX_create()2025-02-13T11:04:20+00:00tb2025-02-13T11:04:20+00:00urn:sha1:6d52d4fe853e27733eac9f76135f5e20b77e61a7
This does what the public BN_MONT_CTX_new() should have done in the first
place rather than doing the toolkit thing of returning an invalid object
that you need to figure out how to populate and with what because the docs
are abysmal.
It takes the required arguments and calls BN_MONT_CTX_set(), which all
callers do immediately after _new() (except for DSA which managed to
squeeze 170 lines of garbage between the two calls).
ok jsing
Rename BN_mod_exp_recp() to BN_mod_exp_reciprocal()2025-02-12T21:21:34+00:00tb2025-02-12T21:21:34+00:00urn:sha1:dc6d44d71f3b088ce53a7010caf810507bf2d721
(leaving out a dotasm comment that would become harder to read than it
already is)
bn_recp: reformat another ugly comment2025-02-04T20:22:20+00:00tb2025-02-04T20:22:20+00:00urn:sha1:2076aae78095fd10e70ad30d488bdf5e9964ca98Inline BN_reciprocal() in its only caller2025-02-04T12:47:58+00:00tb2025-02-04T12:47:58+00:00urn:sha1:b732166c5143f7ab1b657c8c0581ad8e8449ce84
This is simpler, doesn't need an auxiliary function of dubious value,
avouds an auxiliary variable and gets rid of a bunch of comments that
are hard to make sense of.
This doesn't bother to invalidate recp->shift since on error you should
not be reusing the RECP_CTX without reinitializing it.
ok jsing
Start cleaning up BN_div_reciprocal() a bit2025-02-04T05:09:53+00:00tb2025-02-04T05:09:53+00:00urn:sha1:09bef15f6dde7dfa51b7c2b40bd8dc186f44edf8
The fast path where no division is performed can be dealt with without
BN_CTX, so do that up front so there's no need to clean up before return.
Error check BN_CTX_get() on each use asd simplify the logic for optional
input parameters. KNF for an ugly comment.
ok jsing
bn_recp: Avoid complication for negative moduli2025-01-22T12:53:16+00:00tb2025-01-22T12:53:16+00:00urn:sha1:b247738e05b5336b6b755731d3686ac29e6cf54f
Instead of doing a weird dance, set the sign on N in BN_RECP_CTX_create().
Since we're not exposing a general purpose calculator API, we can simplify.
ok jsing