VirtualBox

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#17409 closed defect (fixed)

MODIFYVM_AUDIOOUT in VBoxManageModifyVM.cpp, a copy of MODIFYVM_AUDIOIN -> fixed in 5.2.12

Reported by: Socratis Owned by:
Component: other Version: VirtualBox 5.2.0
Keywords: audio out, settings Cc:
Guest type: all Host type: all

Description

This was brought to my attention from an IRC user (jerryaldrichiii) that when setting "--audioout on", it wasn't working. I verified it that the wrong setting was written in the .vbox file. I was also pointed to a forums topic, "5.2.0 r118431 Sound Problem" which was talking about the same thing.

Digging a little bit more, I found out that the two cases MODIFYVM_AUDIOOUT and MODIFYVM_AUDIOIN in "VBoxManageModifyVM.cpp", seem to be a copy/paste. The line:

CHECK_ERROR(audioAdapter, COMSETTER(EnabledIn)(ValueUnion.f));
                                           ^^

in the MODIFYVM_AUDIOOUT case should be:

CHECK_ERROR(audioAdapter, COMSETTER(EnabledOut)(ValueUnion.f));
                                           ^^^

I put the version as 5.2.0, since according to the forum post, this must have been going on since the 5.2.0 days.

Change History (3)

comment:1 by Socratis, 6 years ago

#17733 is probably a duplicate of this one. Can someone please take a look at this? It's an unrealistically obvious problem / easy resolution.

I mean *I* managed to find the culprit... :D

comment:2 by Michael Thayer, 6 years ago

Resolution: fixed
Status: newclosed
Summary: MODIFYVM_AUDIOOUT in VBoxManageModifyVM.cpp, a copy of MODIFYVM_AUDIOINMODIFYVM_AUDIOOUT in VBoxManageModifyVM.cpp, a copy of MODIFYVM_AUDIOIN -> fixed in 5.2.12

Thanks Socratis!

comment:3 by Socratis, 6 years ago

Thanks Michael, confirmed! Forum post updated as well.

For those interested in the actual changeset: https://www.virtualbox.org/changeset/72143/vbox

Note: See TracTickets for help on using tickets.

© 2023 Oracle
ContactPrivacy policyTerms of Use