bsdinstall(8): partedit crashes when modifying a zfs partition's mountpoint
Needs ReviewPublic
Actions

Authored by will_worrbase.com on Mar 3 2018, 8:47 AM.

Tags

None

Referenced Files

	F82207681: D14574.diff
	Fri, Apr 26, 12:38 PM

	Unknown Object (File)
	Fri, Apr 19, 2:48 PM

	Unknown Object (File)
	Mar 4 2024, 10:06 AM

	Unknown Object (File)
	Dec 23 2023, 1:24 AM

	Unknown Object (File)
	Aug 16 2023, 4:47 AM

	Unknown Object (File)
	Jun 27 2023, 7:12 AM

	Unknown Object (File)
	May 3 2023, 5:01 PM

	Unknown Object (File)
	Feb 22 2023, 7:03 AM

View All 10 Files

Subscribers

imp

Contributor Reviews (src)

This revision needs review, but there are no reviewers specified.

Details

Reviewers: None

Summary

bsdinstall(8): partedit crashes when modifying a zfs partition's
mountpoint

When modifying a ZFS partition to change the mountpoint (or pool name, really),
zpool_name isn't set, causing partedit to crash.

Also removed a useless NULL check.

Test Plan

Tested by modifying an existing ZFS mountpoint in bsdinstall(8)

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 15352
Build 15407: arc lint + arc unit

Event Timeline

will_worrbase.com created this revision.Mar 3 2018, 8:47 AM

Herald added subscribers: Contributor Reviews (src), imp. · View Herald TranscriptMar 3 2018, 8:47 AM

Harbormaster completed remote builds in B15352: Diff 39913.Mar 3 2018, 8:47 AM

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=205828

Revision Contents
Changeset List

Path

Size

usr.sbin/

bsdinstall/

partedit/

gpart_ops.c

16 lines

Diff 39913

View Options

usr.sbin/bsdinstall/partedit/gpart_ops.c

bsdinstall(8): partedit crashes when modifying a zfs partition's mountpointNeeds ReviewPublicActions