aboutsummaryrefslogtreecommitdiff
path: root/contrib
Commit message (Collapse)AuthorAgeFilesLines
* tzcode: Fix time zone change detection.Dag-Erling Smørgrav9 days1-9/+7
| | | | | | | | | | | | | | | | | | | | | | | | | Prior to the 2022g import, tzloadbody() returned -1 on error. Now it returns an errno code. When I updated the time zone change detection logic to match, I improperly returned errno in all cases, which means that if the time zone file has not changed since we last loaded it, tzloadbody() returns a random errno value instead of 0. Fixes: bc42155199b5 MFC after: 1 week Sponsored by: Klara, Inc. Sponsored by: NetApp, Inc. Reviewed by: markj Differential Revision: https://reviews.freebsd.org/D51405 (cherry picked from commit d63ffdd1ef6368407b35d415237b95cc739d8073) tzcode: Add an explicit "the timezone file has changed" case This is required for the WITHOUT_DETECT_TZ_CHANGES case, since there the value being tested is a numeric literal. Fixes: d63ffdd1ef63 ("tzcode: Fix time zone change detection.") (cherry picked from commit 0bf113e9041fe20e8c671fe6b2cca8612dc77b77)
* kyua: Try harder to delete directories.Dag-Erling Smørgrav9 days2-0/+16
| | | | | | | | | | | | | | When recursing into a directory to delete it, start by chmod'ing it to 0700. This fixes an issue where kyua is able to run, but not debug, a test case that creates unwriteable directories, because when debugging it tries (and fails) to delete the directory after the test completes. MFC after: 1 week Sponsored by: Klara, Inc. Reviewed by: igoro Differential Revision: https://reviews.freebsd.org/D51229 (cherry picked from commit 9bf14f2a475e221c48488984dc5a02a4608bb877)
* Merge commit 8ac140f39084 from llvm git (by Younan Zhang):Dimitry Andric9 days3-20/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [Clang][NFCI] Cleanup the fix for default function argument substitution (#104911) (This is one step towards tweaking `getTemplateInstantiationArgs()` as discussed in https://github.com/llvm/llvm-project/pull/102922) We don't always substitute into default arguments while transforming a function parameter. In that case, we would preserve the uninstantiated expression until after, e.g. building up a CXXDefaultArgExpr and instantiate the expression there. For member function instantiation, this algorithm used to cause a problem in that the default argument of an out-of-line member function specialization couldn't get properly instantiated. This is because, in `getTemplateInstantiationArgs()`, we would give up visiting a function's declaration context if the function is a specialization of a member template. For example, ```cpp template <class T> struct S { template <class U> void f(T = sizeof(T)); }; template <> template <class U> void S<int>::f(int) {} ``` The default argument `sizeof(U)` that lexically appears inside the declaration would be copied to the function declaration in the class template specialization `S<int>`, as well as to the function's out-of-line definition. We use template arguments collected from the out-of-line function definition when substituting into the default arguments. We would therefore give up the traversal after the function, resulting in a single-level template argument of the `f` itself. However the default argument here could still reference the template parameters of the primary template, hence the error. In fact, this is similar to constraint checking in some respects: we actually want the "whole" template arguments relative to the primary template, not those relative to the function definition. So this patch adds another flag to indicate `getTemplateInstantiationArgs()` for that. This patch also consolidates the tests for default arguments and removes some unnecessary tests. This fixes a crash or assertion failure while building tests for the devel/hpx port. PR: 288352 MFC after: 3 days (cherry picked from commit ffc5ee0f57d56459df93f4107b9835ae78a546b5)
* wg: ipc: add allowed-ip flags support for FreeBSDKyle Evans12 days1-5/+2
| | | | | | | | | | | | | For $reasons, we can't rely on flags in wireguard-tools for the kernel side of WireGuard. Provide a mapping function that uses flags from the kernel that we're building against and fail the operation if we made it to the end without turning some wg(8) flag off. Signed-off-by: Kyle Evans <kevans@FreeBSD.org> Reviewed by: ivy (previous version), Jason A. Donenfeld, jhb (cherry picked from commit f6d9e22982a1d0354c0199e94ebdb4474eaa74b9)
* Merge commit '2a7e45eef31292cf9dd82caf3346eb2acb5b6225'Kyle Evans12 days8-7/+44
| | | | | | | | Update wireguard-tools to the latest version, which has some stubbed out support for incremental Allowed-IPs updates that we would need to add kernel support for. (cherry picked from commit 137de4b34d45192985e21f6d6163533da547fbac)
* mandoc: Bump version string to 1.14.6s20250613Alexander Ziaee2025-07-141-1/+1
| | | | | | | | | | | The mandoc compiler is still at version 1.14.6, but we have imported several snapshots since the version was bumped. Append a snapshot date string to the version to reflect this. Requested by: wosch Discussed with: bapt (cherry picked from commit af2949cd29ef453a63a6345de396be866138b051)
* mandoc: Vendor import of upstream at 2025-06-13Alexander Ziaee2025-07-144-63/+117
| | | | | | | | | Interesting changes: + Lb is now part of SNYOPSIS, the LIBRARY section is deprecated + Ft now takes multiple args, and no longer should be quoted + Fix a segfaut on malformed Tg markup (cherry picked from commit 4c07abdbacf49e4ba749b8720916d8cd340a50b8)
* mandoc: Vendor import of upstream at 2025-04-08Alexander Ziaee2025-07-1418-151/+304
| | | | | | | | | | | | | | | | | | | Interesting changes: + correct legacy mandoc date typo, reported on our very own bugzilla + improve libmandoc manual + strengthen recommendations to omit macros from title lines in mdoc(7) + improve html5 compliance in html output + improve RFC linking in markdown output + libmandoc and mdoc manuals have been improved + support arithmetic operations in tbl(7) column widths for DocBook + define the St -isoC-2023 macro for referencing the C23 spec Approved by: mhorne (mentor, implicit) Reviewed by: imp Discussed with: adrian, bapt, brooks Closes: https://github.com/freebsd/freebsd-src/pull/1689 (cherry picked from commit 80c12959679ab203459dc20eb9ece3a7328b7de5)
* manuals: Align our tree with upstream C23 macroAlexander Ziaee2025-07-141-0/+2
| | | | | | | | | Switch all instances of the -isoC-2024 macro to -isoC-2023 selected by upstream. Keep -isoC-2024 defined, but deprecated, for backwards compat. MFC after: 3 days (resolved merge conflicts w/ cdefs and memccpy.3) (cherry picked from commit 1b6936e125382523509901a4fc5b2932ed0e742d)
* mdoc.7/MANUAL STRUCTURE: add HARDWAREAlexander Ziaee2025-07-141-0/+7
| | | | | | | | | | | | | | | FreeBSD Release infrastructure has been generating the Hardware Compatibility Notes from this section for some decades. Make this more obvious. This does change the upstream mdoc.7 in a minimal way to document this, but the .Dd wasn't bumped to avoid conflicts with the next mdoc/mandoc import. Reported by: bz (HARDWAREs generating compatibility notes) Reported by: grahamperrin (order listed in fdp-primer/man) Reviewed by: imp Pull Request: https://github.com/freebsd/freebsd-src/pull/1463 (cherry picked from commit d235f19d1bf1e3d9ca31822709464cc6250f44c2)
* mandoc: drop 3p sectionBrooks Davis2025-07-141-1/+0
| | | | | | | | | | The 3p is an OpenBSD thing and this list of sections is otherwise entierly customized to FreeBSD. Reviewed by: imp, bapt Differential Revision: https://reviews.freebsd.org/D47948 (cherry picked from commit 3ca117c6bb1dffaa3542f58339154a7dd166d169)
* mandoc: remove duplicate libstdthreads entryBrooks Davis2025-07-141-1/+0
| | | | (cherry picked from commit 0fd7b4655666fce13c43d4562e5f9e54ad32145e)
* Merge mandoc snapshot 2024-09-22Brooks Davis2025-07-1474-1655/+3213
| | | | (cherry picked from commit c1c95add8c80843ba15d784f95c361d795b1f593)
* contrib/mandoc: add -isoC-2024 and -svid1 to mdoc(7)Robert Clausecker2025-07-141-0/+2
| | | | | | | | | | C23 (ISO/IEC 9899:2024) is not out yet, but will be shortly. SVID1 is needed for the history section of memccpy(3). Reviewed by: imp Differential Revision: https://reviews.freebsd.org/D47284 (cherry picked from commit 7ed159c6cc00adf6f75f9a43024f23509f5b8e5e)
* lyaml: vendor import lua bindings for libyamlBaptiste Daroussin2025-07-0831-0/+7317
| | | | (cherry picked from commit 2bc180ef045e5911cce0cea1c2a139cffd2b577a)
* libyaml: import libyaml vendor version 0.2.5Baptiste Daroussin2025-07-0861-0/+18589
| | | | | | | The yaml parser used in nuageinit is too incomplete, import libyaml in order to be able to use as a complete parser for nuageinit. (cherry picked from commit 0f5c86ddb0257f4b7620f1d8e898289be30b19bf)
* contrib/llvm-project: re-add clang/tools/clang-scan-depsDimitry Andric2025-07-023-2/+1098
| | | | | | | | This is in preparation for adding it as an optional tool in base. MFC after: 1 week (cherry picked from commit 7ed1628066eaf55b86f35af86efe804508201cc8)
* kyua: increase cleanup_timeout to 300sLexi Winter2025-06-291-2/+2
| | | | | | | | | | | | | | | Test cleanup times out after 60 seconds, causing the test to fail if the cleanup hasn't finished in time. This timeout should be configurable, but it currently is not. Several tests (e.g. bridge, pf) are now reaching this limit; as a workaround, increase the timeout to 300 seconds. Reviewed by: igoro, kevans, des Approved by: des (mentor), kevans (mentor) Differential Revision: https://reviews.freebsd.org/D50499 (cherry picked from commit 0e33211f911c906b47cd0c457a86372c4b5daf87)
* contrib/llvm-project: fix clang crash compiling modulesLexi Winter2025-06-297-18/+91
| | | | | | | | | | | | | | | | | clang++ may crash when compiling certain C++20 modules. Backport the fix from LLVM upstream. This fixes LLVM bug 102684: https://github.com/llvm/llvm-project/issues/102684. PR: 287803 MFC after: 3 days Obtained from: https://github.com/llvm/llvm-project/pull/102855 Reviewed by: kevans, dim Approved by: kevans (mentor) Differential Revision: https://reviews.freebsd.org/D51041 (cherry picked from commit 55dfaeae8e9aa95f0b724d90ad2423ca1b623142)
* Don't rely on sys/_types.h including sys/cdefs.hBrooks Davis2025-06-231-0/+2
| | | | | | | | | | | | | | These headers relied in __BEGIN_DECS/__END_DECLS being defined when sys/_types.h was included, but there's not a requirement that this be the case. Reviewed by: imp Exp-run by: antoine (PR 286274) Pull Request: https://github.com/freebsd/freebsd-src/pull/1595 (cherry picked from commit b01e971fd39d10273a387788019457792873e81c) (cherry picked from commit 4f4c465b45d42ed8d2b65c8b4c909246987ac877) (cherry picked from commit 12913e4ddf710e517f3ae00b7b420b783cca9b37)
* libarchive/test: fix build when memcpy() is a macroKyle Evans2025-06-191-34/+34
| | | | | | | | | | | | | | | | | | After importing the latest libarchive into FreeBSD, Shawn Webb @ HardenedBSD noted that the test build is broken when FORTIFY_SOURCE=2 while building the base system. Braced initializer lists are a special case that need some extra fun parentheses when we're dealing with the preprocessor. While it's not a particularly common setup, the extra parentheses don't really hurt readability all that much so it's worth fixing for wider compatibility. This corresponds to libarchive PR #2660 Reported by: Shawn Webb (HardenedBSD) (cherry picked from commit e2a605e7ab7ff83d15b1a0b994223768169b0f1e)
* libarchive: merge from vendor branchMartin Matuska2025-06-19178-2342/+25547
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | libarchive 3.8.1 New features: #2088 7-zip reader: improve self-extracting archive detection #2137 zip writer: added XZ, LZMA, ZSTD and BZIP2 support #2403 zip writer: added LZMA + RISCV BCJ filter #2601 bsdtar: support --mtime and --clamp-mtime #2602 libarchive: mbedtls 3.x compatibility Security fixes: #2422 tar reader: Handle truncation in the middle of a GNU long linkname (CVE-2024-57970) #2532 tar reader: fix unchecked return value in list_item_verbose() (CVE-2025-25724) #2532 unzip: fix null pointer dereference (CVE-2025-1632) #2568 warc: prevent signed integer overflow (CVE-2025-5916) #2584 rar: do not skip past EOF while reading (CVE-2025-5918) #2588 tar: fix overflow in build_ustar_entry (CVE-2025-5917) #2598 rar: fix double free with over 4 billion nodes (CVE-2025-5914) #2599 rar: fix heap-buffer-overflow (CVE-2025-5915) Important bugfixes: #2399 7-zip reader: add SPARC filter support for non-LZMA compressors #2405 tar reader: ignore ustar size when pax size is present #2435 tar writer: fix bug when -s/a/b/ used more than once with b flag #2459 7-zip reader: add POWERPC filter support for non-LZMA compressors #2519 libarchive: handle ARCHIVE_FILTER_LZOP in archive_read_append_filter #2539 libarchive: add missing seeker function to archive_read_open_FILE() #2544 gzip: allow setting the original filename for gzip compressed files #2564 libarchive: improve lseek handling #2582 rar: support large headers on 32 bit systems #2587 bsdtar: don't hardlink negative inode files together #2596 rar: support large headers on 32 bit systems #2606 libarchive: support @-prefixed Unix epoch timestamps as date strings #2634 tar: Support negative time values with pax #2637 tar: Keep block alignment after pax error #2642 libarchive: fix FILE_skip regression #2643 tar: Handle extra bytes after sparse entries #2649 compress: Prevent call stack overflow #2651 iso9660: always check archive_string_ensure return value CVE: CVE-2024-57970, CVE-2025-1632, CVE-2025-25724, CVE-2025-5914, CVE-2025-5915, CVE-2025-5916, CVE-2025-5917, CVE-2025-5918 PR: 286944 (exp-run, main, libarchive 3.8.0) (cherry picked from commit 2e113ef82465598b8c26e0ca415fbe90677fbd47)
* MFC: MFV: less v678.Xin LI2025-06-1060-994/+1893
| | | | (cherry picked from commit 252d6dde57d5dd0184929d1f8fb65e7713f51c6d)
* netcat: Allow service names to be used.Dag-Erling Smørgrav2025-06-092-16/+32
| | | | | | | | | | | | | | Someone should really do a vendor import, but it's non-trivial, as we have local modifications. In the meantime, here's a nine-year-old upstream patch which allows service names to be used instead of port numbers. MFC after: 1 week Obtained from: OpenBSD Reviewed by: allanjude Differential Revision: https://reviews.freebsd.org/D50348 (cherry picked from commit 6d3d1fc3a30453be19831f79399bcba0ae822ad1)
* libpam: Additional module directory.Dag-Erling Smørgrav2025-06-091-0/+2
| | | | | | | | | | | | | | Many if not most ports that install PAM modules install them into ${LOCALBASE}/lib/security instead of just ${LOCALBASE}/lib, because all the world is Linux. Rather than try to fix the ports, add that directory to the search path. MFC after: 1 week Relnotes: yes Reviewed by: delphij Differential Revision: https://reviews.freebsd.org/D50626 (cherry picked from commit bb625fc2ac801d0a06ef0edab37a9bdaf209f008)
* atf: Don't be deterred by weird umasks.Dag-Erling Smørgrav2025-05-314-124/+43
| | | | | | | | | | | | | | If the current umask is weird, ATF goes to great lengths to tell us that it can't possibly work in these conditions, instead of just dealing with it. This makes it unreasonably hard to use ATF to test how our own code handles unusual umasks. MFC after: 1 week Sponsored by: Klara, Inc. Reviewed by: igoro, kevans, ngie Differential Revision: https://reviews.freebsd.org/D50267 (cherry picked from commit 1a6a36a98ad5dc86862541b91686b00cf88e9f76)
* libc++: fix compiling <locale> with -fmodulesJordan Gordeev2025-05-251-1/+7
| | | | | | | | | | | | | | | | | | | In /usr/include/c++/v1/__locale_dir/locale_base_api.h, xlocale.h is included without first including stdio.h and stdlib.h, which causes functions like strtoll_l() or sscanf_l() to not be declared. When compiling with -fmodules, locale_base_api.h is processed separately due to a declaration in /usr/include/c++/v1/module.modulemap, and this will cause errors due to the above undeclared symbols. Meanwhile, upstream has substantially reorganized this part of libc++'s headers, so apply a minimalistic workaround: specifically when compiling with -fmodules, add includes of stdio.h and stdlib.h. PR: 286342 MFC after: 1 week (cherry picked from commit c809b0184d0a6543bc5327d4252fa56a07ce4689)
* traceroute: move from contrib to usr.sbinLexi Winter2025-05-2118-3768/+0
| | | | | | | | | | | | traceroute hasn't had a vendor import since 2002, while since then it's had several significant FreeBSD-specific commits. Since it's unlikely another vendor import will happen, and to make the merge of traceroute6 into traceroute easier, import traceroute into usr.sbin. Reviewed by: imp Pull Request: https://github.com/freebsd/freebsd-src/pull/1100 (cherry picked from commit 9b7a920a12a9377b9c8227f72748ab32fbbb4822)
* wpa_supplicant: Use PF_LOCAL instead of PF_INET*Cy Schubert2025-05-111-12/+2
| | | | | | | | | | | | | | | | | | Using a local socket circumvents the issue of needing any Internet protocol (PF_INET or PF_INET6) support. We only use interface ("i" group) ioctls on this socket so we can get away with using PF_LOCAL instead of requring any Internet protocol (PF_INET or PF_INET6) support on the system. This simplifies the patch from 8c7149c73f8f. PR: 286259 Noted by: kevans Fixes: 8c7149c73f8f Reviewed by: kevans, philip, zarychtam_plan-b.pwste.edu.pl (mzar) Differential revision: https://reviews.freebsd.org/D50050 (cherry picked from commit 6aeec6a741975143f53feabceb1726d5033aafd6)
* wpa_supplicant: Handle systems without INET (legacy IP) supportMarek Zarychta2025-05-111-0/+10
| | | | | | | | | | | | | Currently, wpa_supplicant fails when legacy IP support is disabled in FreeBSD (i.e., the world built with WITHOUT_INET and nooptions INET in the kernel config). The proposed patch allows running wpa_supplicant and connecting to wireless networks without INET support when INET6 is available. Reviewed by: cy, adrian, philip, roy_marples.name Differential Revision: https://reviews.freebsd.org/D49959 (cherry picked from commit 8c7149c73f8f2301369f271c98470b72973b0c01)
* unbound: Vendor import 1.23.0Cy Schubert2025-05-11222-7275/+41990
| | | | | | | | | Release notes at https://nlnetlabs.nl/news/2025/Apr/24/unbound-1.23.0-released/ Merge commit '44bab727dfe28451b777dc9e47db4f748b709182' (cherry picked from commit be771a7b7f4580a30d99e41a5bb1b93a385a119d)
* bsddialog: Correct type for bsddialog_total_progviewEd Maste2025-05-092-4/+4
| | | | | | | | | | | | It was an int, but printed with %lli format. Although it would be reasonable to use a 32-bit int here (i.e., changing the printf format instead) this matches what was done in bsddialog upstream (and now in main). PR: 286655 Reviewed by: asiciliano Sponsored by: The FreeBSD Foundation Differential Revision: https://reviews.freebsd.org/D50261
* MFV: xz 5.8.1.Xin LI2025-05-0781-2104/+7374
| | | | | | PR: bin/286252 (cherry picked from commit 128836d304d93f2d00eb14069c27089ab46c38d4)
* libedit: import vendor snapshot 2025-01-03Baptiste Daroussin2025-04-3026-98/+350
| | | | (cherry picked from commit 136d69caf03bc38de95c4df34c5a683e9ce81bfa)
* telnet: Prevent buffer overflow in the user prompt for SRAJohn Baldwin2025-04-291-4/+12
| | | | | | | | | | | | | | | | | | | | | | The Secure RPC authenticator for telnet prompts the local user for the username to use for authentication. Previously it was using sprintf() into a buffer of 256 bytes, but the username received over the wire can be up to 255 bytes long which would overflow the prompt buffer. Fix this in two ways: First, use snprintf() and check for overflow. If the prompt buffer overflows, fail authentication without prompting the user. Second, add 10 bytes to the buffer size to account for the overhead of the prompt so that a maximally sized username fits. While here, replace a bare 255 in the subsequent telnet_gets call with an expression using sizeof() the relevant buffer. PR: 270263 Reported by: Robert Morris <rtm@lcs.mit.edu> Tested on: CHERI Reviewed by: emaste Differential Revision: https://reviews.freebsd.org/D49832 (cherry picked from commit 5737c2ae06e143e49496df2ab5a64f76d5456012)
* wireguard-tools: ipc: freebsd: use AF_LOCAL for the control socketKyle Evans2025-04-261-1/+1
| | | | | | | | | | | | AF_INET assumes that IPv4 is built into the kernel, but it's completely valid to build a kernel without it. unix(4) sockets, on the other hand, are not-optional in the kernel build. Given that interface ioctls can be invoked on any kind of socket, switch to the safer one to avoid breaking on IPv6-only kernels. Reported and tested by: ivy (cherry picked from commit 0d238bc50d453d7ac29476fa71edd1fc9a5fbbf9)
* contrib/expat: import expat 2.7.1Philip Paeps2025-04-0528-264/+1779
| | | | | | | | | Changes: https://github.com/libexpat/libexpat/blob/R_2_7_1/expat/Changes https://github.com/libexpat/libexpat/blob/R_2_7_0/expat/Changes Security: CVE-2024-8176 (cherry picked from commit fe9278888fd4414abe2d922e469cf608005f4c65)
* netbsd-tests: Update a test case to chase grep symlink handling changesMark Johnston2025-03-311-1/+1
| | | | | | | | | | | | | This test case verifies that grep detects symlink loops when traversing a directory hierarchy. Fixes: fc12c191c087 ("grep: Default to -p instead of -S.") Reviewed by: ngie, jhb Reported by: Jenkins MFC after: 2 weeks Differential Revision: https://reviews.freebsd.org/D46544 (cherry picked from commit a700bef1e4ee3e6f4e1a86a374bf9b4044f69a70)
* contrib/tzdata: import tzdata 2025bPhilip Paeps2025-03-268-23/+123
| | | | | | Changes: https://github.com/eggert/tz/blob/2025b/NEWS (cherry picked from commit d2cccdef30376b7de5643caae1ef035f8e6932f0)
* contrib/googletest: update from 1.14.0 to 1.15.2Enji Cooper2025-03-2574-1022/+1564
| | | | | | | | | | | | | | | | The changes between the two versions can be found in this diff of the two release tags: https://github.com/google/googletest/compare/v1.14.0...v1.15.2 One notable change is that GoogleTest 1.15.x now officially requires C++-14 (1.14.x required C++-11). MFC after: 1 week Differential Revision: https://reviews.freebsd.org/D47197 Merge commit '14f7077fed7d82046bdcbe347004132f08aba886' (cherry picked from commit 5ca8c28cd8c725b81781201cfdb5f9969396f934)
* libibverbs: Extend support of NDR ratesSlava Shwartsman2025-03-223-0/+11
| | | | | | PR: 285305 (cherry picked from commit 5963423232e869b8dbe8e9a65134e92735dfb521)
* ntp: Replace the workaround from 98e34e8e2557 with a patch from upstreamCy Schubert2025-03-202-5/+1
| | | | | | | | 98e34e8e2557 circumvented an upstream patch which caused an IPv6 pool regresson. This patch removes the circumvention and replaces it with an upstream patch planned for the new release of ntp. (cherry picked from commit bc02e655872021595c434850fbcbdb8dd11d4a46)
* ntp: Another patch to address IPv6 pool regressionCy Schubert2025-03-201-1/+1
| | | | | | | | | | 98e34e8e2557 circumvented an upstream patch which caused an IPv6 pool regression. This patch, discussed in https://bugs.ntp.org/show_bug.cgi?id=3958, addresses another unworkable combination of link-local local address with non-link-local server. (cherry picked from commit c1767cf87cb64c25426fd7fe119be283b134509a)
* ntp: ntpd does not connect to NTP server with link local IPv6 addressCy Schubert2025-03-201-1/+1
| | | | | | | | | | | | | | Upstream bug 3943 (https://bugs.ntp.org/show_bug.cgi?id=3943) discusses: Starting with 4.2.8p18 ntp does not synchronize or even connect to the configured NTP server any more. ntp stays in .INIT. state indefinitely and checking the network traffic shows that ntp does not attempt to contact the NTP server. This is regression introduced by the fix for upstream bug 3913. This is a similar bug I reported upstream (ntp bug 3841). (cherry picked from commit 381956e267569031883a516e10446c29490ee41d)
* kyua: Pass unprivileged user config prop to ATF using all known namesIgor Ostapenko2025-03-164-3/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Kyua and ATF speak different naming styles. In this case, the unprivileged user property can be named with underscore on the Kyua side, and with a hyphen on the ATF side. Sometimes it is not obvious which style should be used in which situation. For instance, a test case may require this configuration property being set using require.config. Also, a test case may want to read the property using something like atf_tc_get_config_var(). Which names should be used in these cases? From the perspective of the original code, it is expected to be this: require.config unprivileged-user atf_tc_get_config_var(tc, "unprivileged-user") But, as long as Kyua is the main interface, its users expect to work with kyua.conf(5), which says that it must be named as unprivileged_user (with underscore). As a result, test authors tend to do this instead: require.config unprivileged_user atf_tc_get_config_var(tc, "unprivileged_user") Kyua already has hacks to understand both unprivileged_user and unprivileged-user coming from require.config. And this patch covers the missing second part -- make Kyua pass both names back to ATF as two identical configuration properties named different ways. Reviewed by: ngie, asomers MFC after: 2 weeks Differential Revision: https://reviews.freebsd.org/D49039 (cherry picked from commit 51a8eb6410461c94c8e0f2b59e3417cfb5d7da75)
* kyua: Support require.diskspace ATF metadataIgor Ostapenko2025-03-043-1/+27
| | | | | | | | Reviewed by: ngie MFC after: 1 month Differential Revision: https://reviews.freebsd.org/D48333 (cherry picked from commit 6befd3511105a17decdafbf9d3f55324c83aaea1)
* ntp: NULL pointer deref when create_interface() failsCy Schubert2025-02-281-1/+1
| | | | | | | | | | | | | Fix NULL pointer dereference when create_interface() fails in update_interfaces(). The upstream bug report says a typo was introduced by https://bk.ntp.org/ntp-dev/ntpd/ntp_io.c?PAGE=diffs&REV=66175036PETA6g__fON8oNrjL54Ttw. ep should have been ep2. Fix obtained from upstream bug report. PR: 285065 Upstream bug: https://bugs.ntp.org/show_bug.cgi?id=3939 (cherry picked from commit 628715fdcc9f2226bfe0f4ebe381aaa7761cb6cc)
* kyuafile.5: ATF metadata mapping referenceIgor Ostapenko2025-02-271-1/+44
| | | | | | | | Reviewed by: ngie MFC after: 1 month Differential Revision: https://reviews.freebsd.org/D48190 (cherry picked from commit 68206add1173920065092c06eea40c90bb73b5c5)
* unifdef: Fix collision check when adding symbols.Dag-Erling Smørgrav2025-02-131-6/+5
| | | | | | | | | | | | | findsym() is intended for use while parsing input, so it should not be called from addsym2() or indirectsym(), which are called before any input is processed. MFC after: 1 week Sponsored by: Klara, Inc. Reviewed by: kevans Differential Revision: https://reviews.freebsd.org/D48733 (cherry picked from commit c63af363c2458aebc30c01cd0b93b4b902580019)
* contrib: Remove now-unused cortex-stringsEd Maste2025-02-1139-7676/+0
| | | | | | | | | | | Replaced by arm-optimized-routines in commit 2e8ff4d1d5b9 ("Switch to Arm Optimized Routines for mem* & str*"). Reviewed by: andrew Sponsored by: The FreeBSD Foundation Differential Revision: https://reviews.freebsd.org/D48500 (cherry picked from commit 4a7d6666c5ddd135b2023b4b3b4962770ff2e73b)