Project

General

Profile

Bug #8065

A manually configured filesystem layout with a separate /usr fails to properly setup several aspects of the system at boot time

Added by Jim Pingle about 2 years ago. Updated about 2 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Operating System
Target version:
Start date:
11/07/2017
Due date:
% Done:

100%

Estimated time:
Affected Version:
2.4.x
Affected Architecture:
amd64

Description

Choosing a manual installation with a separate /usr slice (ex: boot, /, /usr, /var, swap) fails to setup various aspects of the firewall. Early in the boot process at source:src/etc/pfSense-rc#L55 an attempt is made to read the product name using a PHP script. PHP is under /usr, which is not mounted at that time. As such, the product name is empty which causes several other parts of pfSense-rc to misfire.

It does not appear to be necessary to read the product name that early. The first thing that needs it is a call to configure ddb, and nothing else until after the disk/slices are all mounted. DDB does not need to be setup that early.

Moving the read of the product name and the call to configure ddb to just after the mount block appears to fix it, as in the attached patch. The patch works for me on test systems here, but I'd like a little more internal review before committing it.

rc-mount-sooner.diff (1.22 KB) rc-mount-sooner.diff Jim Pingle, 11/07/2017 01:00 PM

Associated revisions

Revision 05871043 (diff)
Added by Jim Pingle about 2 years ago

Reorder reading the product name in pfSense-rc so it happens after all filesystems are mounted. Fixes #8065

History

#1 Updated by Jim Pingle about 2 years ago

  • Assignee changed from Renato Botelho to Jim Pingle

Renato says the patch looks good.

Further testing shows no problems:
- Affected UFS system is OK
- Regular UFS system is OK
- ZFS is OK
- SG-1000 is OK
- SG-3100 is OK

Patch will be committed momentarily.

#2 Updated by Jim Pingle about 2 years ago

  • Status changed from New to Feedback
  • % Done changed from 50 to 100

#3 Updated by Jim Pingle about 2 years ago

  • Status changed from Feedback to Resolved

Also available in: Atom PDF