Page MenuHomeFreeBSD

vmm: Bump vmname buffer in struct vm to VM_MAX_NAMELEN + 1
ClosedPublic

Authored by khng on Aug 1 2021, 10:55 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 3, 9:13 PM
Unknown Object (File)
Tue, Dec 3, 4:30 PM
Unknown Object (File)
Oct 1 2024, 7:33 PM
Unknown Object (File)
Sep 30 2024, 5:25 PM
Unknown Object (File)
Sep 29 2024, 2:41 PM
Unknown Object (File)
Sep 28 2024, 11:19 AM
Unknown Object (File)
Sep 25 2024, 1:51 PM
Unknown Object (File)
Sep 25 2024, 5:21 AM

Details

Summary

In hw.vmm.create sysctl handler the maximum length of vm name is
VM_MAX_NAMELEN. However in vm_create() the maximum length allowed is
only VM_MAX_NAMELEN - 1 chars. Bump the length of the internal buffer to
allow the length of VM_MAX_NAMELEN for vm name.

MFC after: 3 days
Sponsored by: The FreeBSD Foundation

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

khng requested review of this revision.Aug 1 2021, 10:55 AM
This revision is now accepted and ready to land.Aug 2 2021, 6:49 AM
jhb added inline comments.
sys/amd64/vmm/vmm.c
177

I suspect this results in some padding now? Perhaps it would be best to move this field below the vcpu[] array at least?

sys/amd64/vmm/vmm.c
177

It does not create padding with default SPECNAMELEN on FreeBSD 12 and 13, but might be the case for other SPECNAMELEN values.