Page MenuHomeFreeBSD
Differential D2299

Add a sanity check to the swap size in zfsboot of bsdinstall
ClosedPublic
Actions

Authored by allanjude on Apr 15 2015, 8:49 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Oct 12, 1:23 AM
Unknown Object (File)
Sat, Oct 11, 11:20 PM
Unknown Object (File)
Thu, Sep 25, 3:11 AM
Unknown Object (File)
Wed, Sep 24, 3:10 AM
Unknown Object (File)
Sun, Sep 21, 5:20 PM
Unknown Object (File)
Sep 12 2025, 10:45 PM
Unknown Object (File)
Aug 1 2025, 8:14 AM
Unknown Object (File)
Jul 30 2025, 9:47 PM
View All 100 Files
Subscribers
imp

Details

Reviewers
dteske
brd
roberto
lattera-gmail.com
Summary

Shawn Webb reported accidently ending up with 8kb instead of 8gb of swap
This is invalid, and obviously not what the user intended to do

Add a sanity check that does not allow less than 100mb of swap (while still allowing no swap)

MFC-After: 1 month (before 10.2)

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

allanjude updated this revision to Diff 4841.Apr 15 2015, 8:49 PM
allanjude retitled this revision from to Add a sanity check to the swap size in zfsboot of bsdinstall.
allanjude updated this object.
allanjude edited the test plan for this revision. (Show Details)
allanjude added reviewers: dteske, roberto, lattera-gmail.com.
Herald added a subscriber: imp. · View Herald TranscriptApr 15 2015, 8:49 PM
roberto accepted this revision.Apr 15 2015, 9:34 PM
roberto edited edge metadata.

Looks reasonable to me.

This revision is now accepted and ready to land.Apr 15 2015, 9:34 PM
brd accepted this revision.May 5 2015, 2:48 AM
brd added a reviewer: brd.
allanjude closed this revision.May 5 2015, 3:24 AM

Committed as rS282443

Revision Contents
Changeset List

PathSize
usr.sbin/
bsdinstall/
scripts/
26 lines

Diff 4841

View Options

usr.sbin/bsdinstall/scripts/zfsboot

Loading...