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)
	Sun, Aug 23, 6:12 AM

	Unknown Object (File)
	Wed, Aug 12, 7:00 PM

	Unknown Object (File)
	Mon, Aug 10, 5:11 PM

	Unknown Object (File)
	Sun, Aug 9, 12:26 PM

	Unknown Object (File)
	Sun, Aug 9, 7:50 AM

	Unknown Object (File)
	Sun, Aug 9, 3:24 AM

	Unknown Object (File)
	Sat, Aug 8, 2:01 AM

	Unknown Object (File)
	Fri, Aug 7, 12:53 PM

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