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)
May 8 2024, 3:14 PM
Unknown Object (File)
May 6 2024, 7:18 AM
Unknown Object (File)
Apr 28 2024, 7:25 AM
Unknown Object (File)
Apr 25 2024, 1:04 AM
Unknown Object (File)
Apr 22 2024, 6:08 PM
Unknown Object (File)
Jan 16 2024, 12:58 PM
Unknown Object (File)
Jan 15 2024, 5:54 PM
Unknown Object (File)
Dec 21 2023, 7:38 PM

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.