Page MenuHomeFreeBSD
Files F160522929

D54428.diff
No OneTemporary
Actions

D54428.diff
View Options

diff --git a/usr.sbin/makefs/tests/makefs_cd9660_tests.sh b/usr.sbin/makefs/tests/makefs_cd9660_tests.sh
--- a/usr.sbin/makefs/tests/makefs_cd9660_tests.sh
+++ b/usr.sbin/makefs/tests/makefs_cd9660_tests.sh
@@ -61,21 +61,23 @@
atf_test_case D_flag
D_flag_body()
{
- atf_skip "makefs crashes with SIGBUS with dupe mtree entries; see FreeBSD bug # 192839"
-
create_test_inputs
- atf_check -o save:$TEST_SPEC_FILE mtree -cp $TEST_INPUTS_DIR
- atf_check -o not-empty \
- $MAKEFS -F $TEST_SPEC_FILE -M 1m $TEST_IMAGE $TEST_INPUTS_DIR
+ create_manifest_file
+
+ # Check that it works
+ atf_check $MAKEFS -M 1m $TEST_IMAGE $TEST_SPEC_FILE
- atf_check cp $TEST_SPEC_FILE spec2.mtree
- atf_check -o save:${TEST_SPEC_FILE}_dupe cat $TEST_SPEC_FILE spec2.mtree
+ # Duplicate entries in the manifest file
+ cp $TEST_SPEC_FILE spec2.mtree
+ cat $TEST_SPEC_FILE spec2.mtree | sort > "${TEST_SPEC_FILE}_dupe"
- atf_check -o not-empty -s not-exit:0 \
- $MAKEFS -F ${TEST_SPEC_FILE}_dupe -M 1m $TEST_IMAGE $TEST_INPUTS_DIR
- atf_check -o not-empty \
- $MAKEFS -D -F ${TEST_SPEC_FILE}_dupe -M 1m $TEST_IMAGE $TEST_INPUTS_DIR
+ # Check that it errors
+ atf_check -e not-empty -s not-exit:0 \
+ $MAKEFS -M 1m $TEST_IMAGE ${TEST_SPEC_FILE}_dupe
+ # Check that it warns
+ atf_check -e not-empty \
+ $MAKEFS -D -M 1m $TEST_IMAGE ${TEST_SPEC_FILE}_dupe
}
atf_test_case F_flag cleanup
diff --git a/usr.sbin/makefs/tests/makefs_ffs_tests.sh b/usr.sbin/makefs/tests/makefs_ffs_tests.sh
--- a/usr.sbin/makefs/tests/makefs_ffs_tests.sh
+++ b/usr.sbin/makefs/tests/makefs_ffs_tests.sh
@@ -74,21 +74,23 @@
atf_test_case D_flag
D_flag_body()
{
- atf_skip "makefs crashes with SIGBUS with dupe mtree entries; see FreeBSD bug # 192839"
-
create_test_inputs
- atf_check -o save:$TEST_SPEC_FILE mtree -cp $TEST_INPUTS_DIR
- atf_check -o not-empty \
- $MAKEFS -F $TEST_SPEC_FILE -M 1m $TEST_IMAGE $TEST_INPUTS_DIR
+ create_manifest_file
- atf_check cp $TEST_SPEC_FILE spec2.mtree
- atf_check -o save:${TEST_SPEC_FILE}_dupe cat $TEST_SPEC_FILE spec2.mtree
+ # Check that it works
+ atf_check -o not-empty $MAKEFS -M 1m $TEST_IMAGE $TEST_SPEC_FILE
- atf_check -o not-empty -s not-exit:0 \
- $MAKEFS -F ${TEST_SPEC_FILE}_dupe -M 1m $TEST_IMAGE $TEST_INPUTS_DIR
- atf_check -o not-empty \
- $MAKEFS -D -F ${TEST_SPEC_FILE}_dupe -M 1m $TEST_IMAGE $TEST_INPUTS_DIR
+ # Duplicate entries in the manifest file
+ cp $TEST_SPEC_FILE spec2.mtree
+ cat $TEST_SPEC_FILE spec2.mtree | sort > "${TEST_SPEC_FILE}_dupe"
+
+ # Check that it errors
+ atf_check -e not-empty -s not-exit:0 \
+ $MAKEFS -M 1m $TEST_IMAGE ${TEST_SPEC_FILE}_dupe
+ # Check that it warns
+ atf_check -e not-empty -o not-empty \
+ $MAKEFS -D -M 1m $TEST_IMAGE ${TEST_SPEC_FILE}_dupe
}
atf_test_case F_flag cleanup
diff --git a/usr.sbin/makefs/tests/makefs_tests_common.sh b/usr.sbin/makefs/tests/makefs_tests_common.sh
--- a/usr.sbin/makefs/tests/makefs_tests_common.sh
+++ b/usr.sbin/makefs/tests/makefs_tests_common.sh
@@ -79,6 +79,14 @@
-p $TEST_MOUNT_DIR $mtree_excludes_arg
}
+create_manifest_file()
+{
+ # Prefer a specification format that is easier to parse (-C)
+ mtree -k "$DEFAULT_MTREE_KEYWORDS" -cp "$TEST_INPUTS_DIR" | \
+ mtree -C > "$TEST_SPEC_FILE"
+ sed -i "" "s|^\.|.${TEST_INPUTS_DIR##$TMPDIR}|g" "$TEST_SPEC_FILE"
+}
+
create_test_dirs()
{
atf_check mkdir -m 0777 -p $TEST_MOUNT_DIR

File Metadata

Mime Type
text/plain
Expires
Fri, Jun 26, 8:16 AM (10 h, 7 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
34350207
Default Alt Text
D54428.diff (3 KB)

Event Timeline