summaryrefslogtreecommitdiff
path: root/src/usr.bin (unfollow)
Commit message (Collapse)AuthorFilesLines
2025-02-04bn_recp: reformat another ugly commenttb1-5/+6
2025-02-04SSL_select_next_proto: fix invalid octal escape by switching to hexadecimaltb1-3/+3
2025-02-04Inline BN_reciprocal() in its only callertb1-36/+10
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
2025-02-04Start cleaning up BN_div_reciprocal() a bittb1-24/+23
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
2025-02-04Error check i2t_ASN1_OBJECT() and tweak warning messagetb1-2/+4
CID 532326 ok djm jsing
2025-02-01Improve detection and handling of alerts in renegotiation regress.jsing1-23/+76
2025-02-01Hook renegotiation regress.jsing1-1/+2
2025-02-01Fix certificate paths.jsing1-4/+4
2025-02-01Add regress coverage for TLS renegotiation.jsing2-0/+560
2025-01-27Mop up RC4_INDEX.jsing14-116/+19
The RC4_INDEX define switches between base pointer indexing and per-byte pointer increment. This supposedly made a huge difference to performance on x86 at some point, however compilers have improved somewhat since then. There is no change (or effectively no change) in generated assembly on a the majority of LLVM platforms and even when there is some change (e.g. aarch64), there is no noticable performance difference. Simplify the (still messy) macros/code and mop up RC4_INDEX. ok tb@
2025-01-27X509_NAME_print(): remove no longer useful length checktb1-4/+1
This was needed to avoid truncation on BIO_write(). With the switch to BIO_printf() in the previous commit this is no longer needed.
2025-01-27X509_NAME_print: NUL-terminate and switch to BIO_printf()tb1-2/+5
This handles the empty string, which ruby-openssl checks. Pointed out by anton
2025-01-27x509_obj.c: be better at sortingtb1-2/+2
2025-01-26x509_obj.c: fix includestb1-4/+5
2025-01-26Rework X509_NAME_print()tb1-33/+65
This is legacy API that we can unexport since nothing uses it directly. Unfortunately we need to keep the functions because there are plenty of things that use it indirectly by passing XN_FLAG_COMPAT to X509_print_ex(). The old implementation parsed the X509_NAME_oneline() output in order to remove the / preceding the (one or two-uppercase letters) name and to insert ", " afterward. This is just stupid in so many ways, not least because there's basically no limit to the garbage that you can stuff into an X.500 name. So rework this and only include the name entries whose short names are one or two letters long. This way, this becomes slightly saner and less fragile. ok jsing
2025-01-26Rewrite X509_NAME_ENTRY_oneline() using CBB and CBStb2-104/+123
This splits the horrid spaghetti into a few relatively straightforward helpers which do one thing at a time. There are still some spectacular dances around ASN1_GENERALSTRING, but let's blame that one on X.500. In brief, X509_NAME_ENTRY_oneline() iterates over the name entries, and writes out a line /name1=value1,/name2=value2,... which you may have seen variations of in issuer or subject output. The name is the short name or the long name or the textual representation of the OID (truncated to 79 characters) and the value is a string where printable ASCII characters are represented as themselves and otherwise as hexadecimal digits preceded by \x. Except for GENERALSTRING, where the four octet representation is shortened to single-octet representation if none of the top three octets in the entire string is populated. It's the mother of all pretty things. But, hey, you could do worse and try to parse this garbage... ok jsing
2025-01-26x509_utl.c: use normal order of internal headerstb1-3/+2
2025-01-25Remove #error if OPENSSL_NO_FOO is definedtb23-115/+23
discussed with jsing
2025-01-25Garbage collect field_type member of the EC methodstb2-6/+2
ok jsing
2025-01-25Promote a few functions from EC API to garbage bintb1-36/+41
EC_GROUP_method_of() and EC_METHOD_get_field_type() only ever used chained together as a convoluted means to retrieve the field type of a group. This is no longer useful since the answer will always be NID_X9_62_prime_field. EC_POINT_method_of(), EC_GROUP{,_have}_precompute_mult(): exposed by one of those expose-everything perl XS modules. ok jsing
2025-01-25Remove now unused internal ec_group_get_field_type()tb2-12/+2
ok jsing
2025-01-25Remove calls to ec_group_get_field_type() from EC_GROUP_cmp()tb1-3/+1
ok jsing
2025-01-25Make EC_KEY_precompute_mult() return 1 directlytb1-2/+2
This hasn't done anything in a long time. Only dovecot uses an unchecked call to this. With this we can remove EC_GROUP_precompute_mult(). ok jsing
2025-01-25Simplify ecpk_print_explicit_parameters()tb1-4/+2
At this point the NID is always NID_X9_62_prime_field, so we can use SN_X9_62_prime_field directly rather than getting the field type from the method and then converting the nid to an sn with OBJ_nid2sn(). ok jsing
2025-01-25Simplify ec_asn1_group2fieldid()tb1-25/+3
The field_type is always NID_X9_62_prime_field, no need to encode and retrieve this from the group method. ok jsing
2025-01-24Remove now unused perlasm script for MD5 on amd64.jsing1-265/+0
2025-01-24Provide a readable assembly implementation for MD5 on amd64.jsing5-10/+246
This appears to be about 5% faster than the current perlasm version on a modern Intel CPU. While here rename md5_block_asm_data_order to md5_block_data_order, for consistency with other hashes. ok tb@
2025-01-24Remove pointless call to EC_GROUP_precompute_mul()tb1-3/+1
2025-01-22ectest: zap stray whitespacetb1-2/+2
2025-01-22ectest: fix misleading indentationtb1-5/+7
2025-01-22ectest: remove unused definestb1-5/+1
2025-01-22ectest: even more lipsticktb1-17/+15
2025-01-22ectest: apply some more lipsticktb1-8/+3
2025-01-22ectest: switch from new + copy to dup. zap some NULL checks before freetb1-42/+15
2025-01-22bn_test: remove random negative dance for bn_div_reciprocal()tb1-3/+1
2025-01-22Use simpler (if a bit weird) dup instead of new + copytb1-5/+2
ok jsing
2025-01-22bn_recp: Avoid complication for negative modulitb2-13/+5
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
2025-01-22Adjust for BN_div_recp() -> BN_div_reciprocal()tb1-3/+3
2025-01-22Rename BN_div_recp() into BN_div_reciprocal()tb2-7/+7
Requested by jsing
2025-01-22Split BN_mod_sqr_reciprocal() out of BN_mod_mul_reciprocal()tb3-23/+19
There's no need for BN_mod_mul_reciprocal() to have this complication. The caller knows when x == y, so place the burden on the caller. This simplifies both the caller side and the implementation in bn_recp.c. ok jsing
2025-01-22Remove the mul_generator_ct function pointer from EC_METHOD.jsing3-24/+8
There's no need for a separate mul_generator_ct() function pointer - we really only need mul_single_ct() and mul_double_nonct(). And rather than calling ec_mul_ct() and having it figure out which point to use, explicitly pass the generator point when calling mul_single_ct(). ok tb@
2025-01-22Expand the SM4_ROUNDS macro.jsing1-25/+83
This macro references variable names that are in the consuming function and are not actually passed to the macro. Expanding it makes the logic clearer. If we wanted to reduce code the middle six group of rounds could be implemented using a for loop (which the compiler can then choose to unroll). ok tb@
2025-01-22Replace {load,store}_u32_be() with crypto_{load,store}_be32toh().jsing1-39/+25
load_u32_be() and store_u32_be() are not symmetrical, with load_u32_be() having a rather unexpected indexing interface. Fix up the callers to perform their own indexing and use crypto_{load,store}_be32toh() instead. ok tb@
2025-01-22Pull the family key and constant key tables out of SM4_set_key().jsing1-28/+27
ok tb@
2025-01-22BN_mod_mul_reciprocal: remove y == NULL complicationtb1-14/+11
No caller ever passes y == NULL, so remove the corresponding contortions and unindent the relevant bits. ok jsing
2025-01-22Replace rotl() with crypto_rol_u32().jsing1-15/+10
ok tb@
2025-01-22Ensure the cyclic subgroup cycles in the expected number of iterations.jsing1-9/+14
Also print the iteration number and fix some indentation.
2025-01-22ectest: heed long forgotten XXX and switch back to BN_one()tb1-3/+2
2025-01-21EC_GROUP_check(): use accessor rather than reaching into the grouptb1-3/+4
The API will be removed soon. This prepares moving it to its only consumer.
2025-01-21ec/Makefile: remove excess tabstb1-9/+9