Remove unused includes.
DIRDEPS_BUILD: Update dependencies.Sponsored by: Dell EMC Isilon
usr.sbin: normalize paths using SRCTOP-relative paths or :H when possibleThis simplifies make logic/outputMFC after: 1 monthSponsored by: Dell EMC Isilon
Fix usr.sbin/rpcbind ATF tests on 32-bit platformsusr.sbin/rpcbind/tests/addrmerge_test.c Fix some sizeof calculations that work only by luck on 64-bit platforms.PR: 210314Reviewed by: ngieA
Fix usr.sbin/rpcbind ATF tests on 32-bit platformsusr.sbin/rpcbind/tests/addrmerge_test.c Fix some sizeof calculations that work only by luck on 64-bit platforms.PR: 210314Reviewed by: ngieApproved by: re (kib)Reported by: Mark MillardMFC after: 4 weeksSponsored by: Spectra Logic CorpDifferential Revision: https://reviews.freebsd.org/D6855
show more ...
Merge ^/user/ngie/release-pkg-fix-tests to unbreak how test files are installedafter r298107Summary of changes:- Replace all instances of FILES/TESTS with ${PACKAGE}FILES. This ensures that na
Merge ^/user/ngie/release-pkg-fix-tests to unbreak how test files are installedafter r298107Summary of changes:- Replace all instances of FILES/TESTS with ${PACKAGE}FILES. This ensures that namespacing is kept with FILES appropriately, and that this shouldn't need to be repeated if the namespace changes -- only the definition of PACKAGE needs to be changed- Allow PACKAGE to be overridden by callers instead of forcing it to always be `tests`. In the event we get to the point where things can be split up enough in the base system, it would make more sense to group the tests with the blocks they're a part of, e.g. byacc with byacc-tests, etc- Remove PACKAGE definitions where possible, i.e. where FILES wasn't used previously.- Remove unnecessary TESTSPACKAGE definitions; this has been elided into bsd.tests.mk- Remove unnecessary BINDIRs used previously with ${PACKAGE}FILES; ${PACKAGE}FILESDIR is now automatically defined in bsd.test.mk.- Fix installation of files under data/ subdirectories in lib/libc/tests/hash and lib/libc/tests/net/getaddrinfo- Remove unnecessary .include <bsd.own.mk>s (some opportunistic cleanup)Document the proposed changes in share/examples/tests/tests/... via examplesso it's clear that ${PACKAGES}FILES is the suggested way forward in terms ofreplacing FILES. share/mk/bsd.README didn't seem like the appropriate methodof communicating that info.MFC after: never probablyX-MFC with: r298107PR: 209114Relnotes: yesTested with: buildworld, installworld, checkworld; buildworld, packageworldSponsored by: EMC / Isilon Storage Division
DIRDEPS_BUILD: Connect MK_TESTS.Sponsored by: EMC / Isilon Storage Division
Avoid adding the '-release' suffix to non-debug kernel packages.Sponsored by: The FreeBSD Foundation
Fix Coverity warnings regarding r293229rpcbind/check_bound.c Fix CID1347798, a memory leak in mergeaddr.rpcbind/tests/addrmerge_test.c Fix CID1347800 through CID1347803, memory leaks in ATF tes
Fix Coverity warnings regarding r293229rpcbind/check_bound.c Fix CID1347798, a memory leak in mergeaddr.rpcbind/tests/addrmerge_test.c Fix CID1347800 through CID1347803, memory leaks in ATF tests. They are harmless because each ATF test case runs in its own process, but they are trivial to fix. Fix a few other leaks that Coverity didn't detect, too.Coverity CID: 1347798, 1347800, 1347801, 1347802, 1347803MFC after: 2 weeksX-MFC-With: 293229Sponsored by: Spectra Logic Corp
"source routing" in rpcbindFix a bug in rpcbind for multihomed hosts. If the server had interfaces ontwo separate subnets, and a client on the first subnet contacted rpcbind atthe address on the
"source routing" in rpcbindFix a bug in rpcbind for multihomed hosts. If the server had interfaces ontwo separate subnets, and a client on the first subnet contacted rpcbind atthe address on the second subnet, rpcbind would advertise addresses on thefirst subnet. This is a bug, because it should prefer to advertise theaddress where it was contacted. The requested service might be firewalledoff from the address on the first subnet, for example.usr.sbin/rpcbind/check_bound.c If the address on which a request was received is known, pass that to addrmerge as the clnt_uaddr parameter. That is what addrmerge's comment indicates the parameter is supposed to mean. The previous behavior is that clnt_uaddr would contain the address from which the client sent the request.usr.sbin/rpcbind/util.c Modify addrmerge to prefer to use an IP that is equal to clnt_uaddr, if one is found. Refactor the relevant portion of the function for clarity, and to reduce the number of ifdefs.etc/mtree/BSD.tests.distusr.sbin/rpcbind/tests/Makefileusr.sbin/rpcbind/tests/addrmerge_test.c Add unit tests for usr.sbin/rpcbind/util.c:addrmerge.usr.sbin/rpcbind/check_bound.cusr.sbin/rpcbind/rpcbind.husr.sbin/rpcbind/util.c Constify some function argumentsReviewed by: impMFC after: 4 weeksSponsored by: Spectra Logic CorpDifferential Revision: https://reviews.freebsd.org/D4690