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

	Unknown Object (File)
	Tue, Nov 25, 10:32 PM

	Unknown Object (File)
	Sat, Nov 15, 1:02 PM

	Unknown Object (File)
	Wed, Nov 12, 2:00 AM

	Unknown Object (File)
	Sun, Nov 9, 8:29 PM

	Unknown Object (File)
	Mon, Nov 3, 11:49 PM

	Unknown Object (File)
	Sun, Nov 2, 2:09 PM

	Unknown Object (File)
	Oct 26 2025, 12:42 AM

	Unknown Object (File)
	Oct 25 2025, 3:28 AM

View All 94 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