Summary: | lib/libc/gen/basedirname_test:dirname_posix fails: "Input buffer was modified." | ||
---|---|---|---|
Product: | Base System | Reporter: | Enji Cooper <ngie> |
Component: | bin | Assignee: | Ed Schouten <ed> |
Status: | Closed FIXED | ||
Severity: | Affects Some People | CC: | emaste, ngie |
Priority: | --- | Flags: | ngie:
mfc-stable11-
ngie: mfc-stable10- |
Version: | CURRENT | ||
Hardware: | Any | ||
OS: | Any |
Description
Enji Cooper
2016-08-15 17:24:13 UTC
A commit references this bug: Author: ed Date: Mon Aug 15 19:05:42 UTC 2016 New revision: 304175 URL: https://svnweb.freebsd.org/changeset/base/304175 Log: Disable tests for non-standard behaviour of dirname(3)/basename(3). The NetBSD ATF tests explicitly check that these functions do not modify their input. These tests are NetBSD-specific. They test for something that is not part of POSIX. PR: 211873 Reviewed by: ngie Differential Revision: https://reviews.freebsd.org/D7506 Changes: head/contrib/netbsd-tests/lib/libc/gen/t_basedirname.c No rush on the MFCs... I can commit the changes if desired. Reopening and putting in-progress for necessary MFC work. This change doesn't need to be MFCed, right? The changes to basename()/dirname() themselves are not going to be merged back either. (In reply to Ed Schouten from comment #4) Not much is lost in merging this change. It helps downstream consumers if they decide to take the basename/dirname API change. Well, I think it actually makes sense not to MFC it. The changes I'm making to basename(3)/dirname(3) should not be MFCed under any circumstance. FreeBSD<=11 should simply stick to the previously existing semantics. By not merging back the changes to these tests, we can be sure this won't happen. Both in stable/11 and in trunk, we are testing exactly the behaviour that we expect. A commit references this bug: Author: ngie Date: Thu Feb 9 09:02:46 UTC 2017 New revision: 313471 URL: https://svnweb.freebsd.org/changeset/base/313471 Log: MFC r288444,r296586,r301753,r303840,r304175: r288444 (by andrew): Pass 8 arguments to makecontext on arm64 as this is all we support. Obtained from: EuroBSDCon Devsummit r296586 (by bdrewery): Fix and connect setjmp test. r301753: Fix up r274061 Detect /usr/share/dict/words the "right way" by using require.files instead of the hacked up attempt in the dict(..) function, which didn't work properly on systems where MK_DICT == no. r303840 (by jhb): Add timer_settime tests using SIGEV_THREAD. Note that these tests should work fine on NetBSD and other systems as SIGEV_THREAD is POSIX. r304175 (by ed): Disable tests for non-standard behaviour of dirname(3)/basename(3). The NetBSD ATF tests explicitly check that these functions do not modify their input. These tests are NetBSD-specific. They test for something that is not part of POSIX. PR: 211873 Changes: _U projects/stable-10-backport-test-changes/ projects/stable-10-backport-test-changes/contrib/netbsd-tests/lib/libc/db/t_db.sh projects/stable-10-backport-test-changes/contrib/netbsd-tests/lib/libc/gen/t_basedirname.c projects/stable-10-backport-test-changes/contrib/netbsd-tests/lib/libc/setjmp/t_setjmp.c projects/stable-10-backport-test-changes/contrib/netbsd-tests/lib/libc/setjmp/t_threadjmp.c projects/stable-10-backport-test-changes/contrib/netbsd-tests/lib/libc/sys/t_getcontext.c projects/stable-10-backport-test-changes/contrib/netbsd-tests/lib/libc/sys/t_timer_create.c projects/stable-10-backport-test-changes/etc/mtree/BSD.tests.dist projects/stable-10-backport-test-changes/lib/libc/tests/Makefile |