Differential D38900

bhyvectl: nvlist leak at send_message()
ClosedPublic
Actions

Authored by gusev.vitaliy_gmail.com on Mar 3 2023, 8:36 PM.

Tags

Referenced Files

	F107140774: D38900.diff
	Fri, Jan 10, 6:14 PM

	F107112540: D38900.diff
	Fri, Jan 10, 7:30 AM

	Unknown Object (File)
	Sat, Jan 4, 8:24 AM

	Unknown Object (File)
	Oct 29 2024, 10:29 AM

	Unknown Object (File)
	Oct 7 2024, 6:16 AM

	Unknown Object (File)
	Sep 25 2024, 4:03 PM

	Unknown Object (File)
	Sep 25 2024, 4:03 PM

	Unknown Object (File)
	Sep 25 2024, 4:03 PM

View All 64 Files

Subscribers

Details

Reviewers

rew
jhb
markj
corvink

Group Reviewers

Commits

rGf63fd20a9ac2: bhyvectl: don't leak nvlist in send_message
rG942525ab47e9: bhyvectl: don't leak nvlist in send_message

Summary

If error occurs, nvlist_destroy() is not called.

Sponsored by: vStack

Test Plan

Simple patch of series. Just compile and verify that "suspend" command works.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

gusev.vitaliy_gmail.com created this revision.Mar 3 2023, 8:36 PM

Herald added a reviewer: bhyve. · View Herald TranscriptMar 3 2023, 8:36 PM

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

gusev.vitaliy_gmail.com requested review of this revision.Mar 3 2023, 8:36 PM

gusev.vitaliy_gmail.com added a parent revision: D38889: bhyvectl: Correct socket_fd closing in send_message().

gusev.vitaliy_gmail.com added a child revision: D38866: libvmm: Add missed ioctl to vm_ioctl_cmds.

markj accepted this revision as: markj.Mar 3 2023, 8:54 PM

This revision is now accepted and ready to land.Mar 3 2023, 8:54 PM

corvink accepted this revision.Mar 6 2023, 11:33 AM

Closed by commit rG942525ab47e9: bhyvectl: don't leak nvlist in send_message (authored by gusev.vitaliy_gmail.com, committed by corvink). · Explain WhyMar 6 2023, 1:06 PM

This revision was automatically updated to reflect the committed changes.

corvink added a commit: rG942525ab47e9: bhyvectl: don't leak nvlist in send_message.

corvink added a commit: rGf63fd20a9ac2: bhyvectl: don't leak nvlist in send_message.Mar 17 2023, 10:31 AM

Revision Contents
Changeset List

Path

Size

usr.sbin/

bhyvectl/

2 lines

Diff 118282

usr.sbin/bhyvectl/bhyvectl.c

Loading...