Differential D34291

bhyve/snapshot: limit snapshot filename to NAME_MAX
ClosedPublic
Actions

Authored by rew on Feb 15 2022, 11:08 PM.

Tags

None

Referenced Files

	F168019926: D34291.id103977.diff
	Tue, Aug 25, 11:09 PM

	F167986877: D34291.id.diff
	Tue, Aug 25, 6:46 PM

	F167929463: D34291.diff
	Tue, Aug 25, 11:48 AM

	Unknown Object (File)
	Tue, Aug 18, 6:57 AM

	Unknown Object (File)
	Mon, Aug 17, 1:44 PM

	Unknown Object (File)
	Mon, Aug 17, 1:40 PM

	Unknown Object (File)
	Mon, Aug 17, 1:39 PM

	Unknown Object (File)
	Sat, Aug 15, 8:55 PM

Subscribers

Details

Reviewers

Group Reviewers

Commits

rG4b905478ffe4: bhyve/snapshot: limit snapshot filename to NAME_MAX
rG961e6a12efdb: bhyve/snapshot: limit snapshot filename to NAME_MAX

Summary

NAME_MAX is a better fit since strcat_extension() constructs the
filename of the snapshot file.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

rew created this revision.Feb 15 2022, 11:08 PM

Herald added a reviewer: bhyve. · View Herald TranscriptFeb 15 2022, 11:08 PM

Herald added subscribers: bcran, rgrimes, imp. · View Herald Transcript

rew requested review of this revision.Feb 15 2022, 11:08 PM

Harbormaster completed remote builds in B44470: Diff 102814.Feb 15 2022, 11:08 PM

rew added a parent revision: D34290: libvmm: use a const char pointer for vm name in struct vmctx.Feb 15 2022, 11:08 PM

rew added a child revision: D34292: bhyve: sweep MAX_VMNAME.

markj accepted this revision as: markj.Feb 28 2022, 2:50 PM

This revision is now accepted and ready to land.Feb 28 2022, 2:50 PM

Closed by commit rG961e6a12efdb: bhyve/snapshot: limit snapshot filename to NAME_MAX (authored by rew). · Explain WhyMar 18 2022, 5:54 AM

This revision was automatically updated to reflect the committed changes.

rew added a commit: rG961e6a12efdb: bhyve/snapshot: limit snapshot filename to NAME_MAX.

jhb added a commit: rG4b905478ffe4: bhyve/snapshot: limit snapshot filename to NAME_MAX.Jan 26 2023, 6:56 PM

Revision Contents
Changeset List

Path

Size

usr.sbin/

bhyve/

6 lines

Diff 103977

usr.sbin/bhyve/snapshot.c

Loading...