OpenBSD src changes summary for 2017-01-06 ==========================================
distrib/sets gnu/usr.bin/clang include/stddef.h lib/libcrypto regress/usr.bin share/man sys/arch/arm/arm sys/arch/arm/include sys/arch/arm/mainbus sys/arch/loongson/loongson sys/arch/octeon/octeon sys/arch/sgi/sgi sys/net sys/netinet sys/netinet6 sys/sys usr.bin/mandoc usr.bin/ssh usr.bin/tmux == distrib =========================================================== 01/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/distrib sets ~ lists/comp/mi > sync (deraadt@) == gnu =============================================================== 02/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/gnu usr.bin/clang ~ include/clang/intrin/Makefile > Don't install standard C headers for which we already have a proper system > version. Also don't install vadefs.h which is a microsoft invention. > ok patrick@ (kettenis@) == include =========================================================== 03/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/include stddef.h ~ stddef.h > Add max_align_t. > ok millert@ (kettenis@) == lib =============================================================== 04/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/lib libcrypto ~ man/d2i_ASN1_OCTET_STRING.3 > tweak previous; (jmc@) ~ man/BIO_s_bio.3 > delete bogus cross references reported by jmc@ > and add some missing escaping of backslashes while here (schwarze@) ~ man/SMIME_read_PKCS7.3 > Delete bogus cross reference reported by jmc@. > Documenting these trivial PKCS7_type_is_*() macros > does not seem useful, at least not right now. (schwarze@) ~ man/EVP_PKEY_sign.3 ~ man/EVP_PKEY_verify_recover.3 > fix typos in cross references reported by jmc@ (schwarze@) ~ man/X509_NAME_add_entry_by_txt.3 > Delete a sentence that attempted to explain an implementation detail > by referencing a non-existent manual page. > Broken .Xr reported by jmc@. (schwarze@) ~ man/BIO_f_base64.3 > Delete a cross reference to the non-existent manual page BIO_set_flags(3), > reported by jmc@. Documenting that function would be a bad idea. All > other flags are used internally and should better not be tampered with. > It looks like an internal function that was made public by mistake, > then abused for an unrelated user interface purpose: a classic case > of botched user interface design. > Instead, only show how to use this function for this one specific purpose. > While here, delete a sentence from the DESCRIPTION that merely > duplicated content from the BUGS section. (schwarze@) ~ man/des_read_pw.3 > Add EVP_read_pw_string(3) to NAME and SYNOPSIS, > resolving a dangling cross reference reported by jmc@. > Sort NAME and SYNOPSIS to agree with .Dt and DESCRIPTION. > Unify parameter names. > Delete a sentence about an implementation detail that is no longer true. > Mention the length limitation of the *_string() variants. (schwarze@) ~ man/HMAC.3 > Remove a dangling cross reference reported by jmc@. > I'm not convinced documenting EVP_MD_CTX_set_flags(3) would be wise. > Instead, refer people to the header file to make it more obvious > that they are tinkering with internals when using such flags. (schwarze@) ~ man/PKCS7_sign_add_signer.3 > Remove a dangling .Xr to PKCS7_SIGNER_INFO_sign(3) reported by jmc@ > and just use .Fn for now. > There are about two dozen interfaces dealing with PKCS7_SIGNER_INFO > objects and none but the constructor, destructor, decoder, and encoder > are documented so far. It makes no sense to document one random one, > and i'm not going to document all of PKCS7_SIGNER_INFO right now. > (schwarze@) ~ man/PKCS7_sign.3 ~ man/PKCS7_sign_add_signer.3 > Remove dangling .Xrs to PKCS7_final(3) reported by jmc@ > and just use .Fn for now. > Not counting constructors, destructors, decoders, encoders, and > debuggers, six out of 24 public functions operating on PKCS7 objects > are currently documented. I'm not documenting the remaining 18 ones > at this point in time. (schwarze@) ~ man/UI_new.3 > Remove bogus cross reference to ui_create(3) reported by jmc@ > and refer readers to the header file instead. > I'm not convinced customized prompting is such a bright idea, it > feels somewhat like overengineering, so i'm not documenting it right > now. People who really feel compelled to roll their own prompting > can go read the source code. (schwarze@) ~ man/X509_check_ca.3 > Delete a cross reference to the undocumented function X509_check_purpose(3) > that wasn't accompanied by any related information. Reported by jmc@. > There are a dozen functions handling X509_PURPOSE objects, all > undocumented, a host of defines, and it seems that a callback is > required. So this seems complicated, i doubt that is much used > in practice, and i'm not diving into it at this point in time. (schwarze@) ~ man/X509_CRL_get0_by_serial.3 > Replace two dangling .Xrs to sk_*() macros with .Fn; reported by jmc@. > The safestack stuff is the most ill-designed user interface i have > seen so far in OpenSSL. It looks positively undocumentable. > At least i'm not trying to document it right now. (schwarze@) ~ man/BIO_get_ex_new_index.3 ~ man/CRYPTO_set_ex_data.3 ~ man/ECDSA_SIG_new.3 ~ man/RSA_get_ex_new_index.3 ~ man/engine.3 > resolve duplication of names and prototypes in manuals related to ex_data > and sprinkle cross references instead; more work is obviously needed here > (schwarze@) ~ man/X509_VERIFY_PARAM_set_flags.3 > Delete a sentence containing a cross reference to an undocumented > function that had the the sole purpose of discouraging its use. > Not talking about it at all discourages using it even more. > Dangling cross reference reported by jmc@. (schwarze@) ~ man/X509_LOOKUP_hash_dir.3 ~ man/X509_STORE_set1_param.3 > Remove cross references to the undocumented functions X509_STORE_new(3) > and X509_STORE_add_lookup(3) reported by jmc@. > Even though these functions are public, they seem more useful internally > than for application programs, so now is not the time to document them. > (schwarze@) == regress =========================================================== 05/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/regress usr.bin ~ ssh/forwarding.sh > Make forwarding test less racy by using unix domain sockets instead of > TCP ports where possible. Patch from cjwatson at debian.org via bz#2659. > (dtucker@) ~ ssh/integrity.sh > Account for timeouts in the integrity tests as failures. > If the first test in a series for a given MAC happens to modify the low > bytes of a packet length, then ssh will time out and this will be > interpreted as a test failure. Patch from cjwatson at debian.org via > bz#2658. (dtucker@) ~ ssh/agent-getpeereid.sh > use correct ssh-add program; bz#2654, from Colin Watson (djm@) == share ============================================================= 06/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/share man ~ man4/malo.4 ~ man4/urtw.4 > No point in having an .Xr hostapd if hostap mode is not supported by the > driver, so comment it out as is done elsewhere. > ok stsp (tb@) ~ man5/ruby-module.5 > Add ruby 2.4 information (jeremy@) == sys =============================================================== 07/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sys arch/arm/arm ~ bcopyinout.S ~ bus_space_asm_generic.S ~ copystr.S ~ cpufunc.c ~ fault.c ~ irq_dispatch.S ~ locore.S > unifdef CPU_ARMv7 and ARM_ARCH_7 > ok kettenis@ patrick@ (jsg@) arch/arm/include ~ atomic.h ~ cpu.h ~ cpuconf.h ~ cpufunc.h ~ frame.h ~ pmap.h > unifdef CPU_ARMv7 and ARM_ARCH_7 > ok kettenis@ patrick@ (jsg@) ~ endian.h > there is no longer a need to ifdef __armv7__ armv6 rev instructions (jsg@) arch/arm/mainbus ~ mainbus.c > unifdef CPU_ARMv7 and ARM_ARCH_7 > ok kettenis@ patrick@ (jsg@) arch/loongson/loongson ~ machdep.c > Ansify cpu_sysctl() on mips64 platforms. > OK patrick@, visa@, jasper@, mpi@ (fcambus@) arch/octeon/octeon ~ machdep.c > Ansify cpu_sysctl() on mips64 platforms. > OK patrick@, visa@, jasper@, mpi@ (fcambus@) arch/sgi/sgi ~ machdep.c > Ansify cpu_sysctl() on mips64 platforms. > OK patrick@, visa@, jasper@, mpi@ (fcambus@) net ~ if_var.h > Remove the global viftable vector that holds the virtual interfaces > configuration and instead use ifnet to store the configuration and > counters. With this we can safely use multicast routing daemons on > multiple domains without vif id colisions. > ok mpi@ (rzalamena@) netinet ~ ip_mroute.c > Kill various splsoftnet(). > ok rzalamena@, visa@ (mpi@) ~ ip_mroute.c > Simplify code by removing some old pullup macro, killing some variables > and using m_dup_pkt() instead of m_copym() with max_linkhdr space adjust > on packet sending to avoid more mbuf allocations. > with input from millert@ and mikeb@, > ok mikeb@ (rzalamena@) ~ ip_mroute.c ~ ip_mroute.h > Remove the global viftable vector that holds the virtual interfaces > configuration and instead use ifnet to store the configuration and > counters. With this we can safely use multicast routing daemons on > multiple domains without vif id colisions. > ok mpi@ (rzalamena@) netinet6 ~ ip6_mroute.c > Kill various splsoftnet(). > ok rzalamena@, visa@ (mpi@) sys ~ cdefs.h > Add C11 support. > ok millert@ (kettenis@) == usr.bin =========================================================== 08/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.bin mandoc ~ mandoc.1 > sort options list; the same as jmc@ did in man.1 rev. 1.18 (schwarze@) ssh ~ sshd_config.5 > Re-add '%k' token for AuthorizedKeysCommand which was lost during the > re-org in rev 1.235. bz#2656, from jboning at gmail.com. (dtucker@) ~ ssh-keyscan.c > Avoid confusing error message when attempting to use ssh-keyscan built > without SSH protocol v.1 to scan for v.1 keys; bz#2583 (djm@) ~ servconf.c > sshd_config is documented to set GSSAPIStrictAcceptorCheck=yes by > default, so actually make it do this. bz#2637 ok dtucker (djm@) ~ readconf.c > show a useful error message when included config files can't be > opened; bz#2653, ok dtucker@ (djm@) ~ readconf.c > fix previous (djm@) ~ sshd_config.5 > keep the tokens list sorted; (jmc@) tmux ~ cmd-command-prompt.c ~ cmd-confirm-before.c ~ cmd.c ~ key-bindings.c ~ status.c ~ tmux.1 ~ tmux.h ~ window-copy.c > Incremental search in copy mode (on for emacs keys by default) - much > the same as normal searching but updates the cursor position and marked > search terms as you type. C-r and C-s in the prompt repeat the search, > once finished searching (with Enter), N and n work as before. (nicm@) ~ cmd-load-buffer.c ~ cmd-save-buffer.c ~ cmd-command-prompt.c ~ cmd-switch-client.c > Nits found with clang. (nicm@) =============================================================================== _______________________________________________ odc mailing list [email protected] http://www.squish.net/mailman/listinfo/odc
