Differential D23400

geli: add a test case for attaching multiple providers with 1 command
ClosedPublic
Actions

Authored by asomers on Jan 28 2020, 5:06 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Jan 11 2024, 10:45 PM

	Unknown Object (File)
	Jan 10 2024, 6:34 PM

	Unknown Object (File)
	Dec 29 2023, 5:58 PM

	Unknown Object (File)
	Dec 20 2023, 2:55 AM

	Unknown Object (File)
	Dec 15 2023, 9:20 PM

	Unknown Object (File)
	Nov 6 2023, 4:21 PM

	Unknown Object (File)
	Sep 15 2023, 5:27 PM

	Unknown Object (File)
	Jul 6 2023, 7:59 AM

View All 23 Files

Subscribers

Details

Reviewers

Commits

rS357880: MFC r357211:
rS357211: geli: add a test case for attaching multiple providers with 1 command

Summary

geli: add a test case for attaching multiple providers with 1 command

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

asomers created this revision.Jan 28 2020, 5:06 PM

Herald added a subscriber: imp. · View Herald TranscriptJan 28 2020, 5:06 PM

Harbormaster completed remote builds in B28991: Diff 67423.Jan 28 2020, 5:06 PM

LGTM. 8kB is overkill for the key, but mostly harmless.

tests/sys/geom/class/eli/attach_test.sh
85 ↗	(On Diff #67423)	I don't know what geli does with it, but 32 bytes (256 bits) is completely sufficient as a key. I checked and geli doesn't reject 256 bit keys. Unfortunately, it does accept keys smaller than 256 bits. It also accepts keys smaller than 128 bits :-(. This is probably a bug we should fix (orthogonal to this test). I filed PR 243676.

This revision is now accepted and ready to land.Jan 28 2020, 5:25 PM

Closed by commit rS357211: geli: add a test case for attaching multiple providers with 1 command (authored by asomers). · Explain WhyJan 28 2020, 6:32 PM

This revision was automatically updated to reflect the committed changes.

asomers added a commit: rS357211: geli: add a test case for attaching multiple providers with 1 command.

asomers added a commit: rS357880: MFC r357211:.Feb 13 2020, 8:46 PM

Revision Contents
Changeset List

Path

Size

head/

tests/

sys/

geom/

class/

eli/

31 lines

Diff 67428

head/tests/sys/geom/class/eli/attach_test.sh

Loading...