USBProxyServiceLinux.cpp

Timestamp:

Oct 26, 2007 5:28:37 PM (17 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

25743

Message:

Main: Fixed memory errors and leaks found by valgrind.

File:

trunk/src/VBox/Frontends/VBoxBFE/USBProxyServiceLinux.cpp

r4071	r5523
840	840	{
841	841	USBINTERFACE If = {0};
	842	bool fIfAdopted = false;
842	843	while (*psz && VBOX_SUCCESS (rc))
843	844	{
…	…
883	884	Assert (!If.bInterfaceNumber); Assert (!If.bAlternateSetting);
884	885	*pIf = If;
	886	fIfAdopted = true;
885	887	}
886	888	else
…	…
895	897	pIf = &pCfg->paInterfaces[If.bInterfaceNumber];
896	898	if (!If.bAlternateSetting)
	899	{
	900	freeInterfaceMembers (pIf, 1);
897	901	*pIf = If;
	902	fIfAdopted = true;
	903	}
898	904	else
899	905	{
…	…
902	908	{
903	909	pIf->paAlts = paAlts;
904		pIf = &paAlts[pIf->cAlts++];
	910	// don't do pIf = &paAlts[pIf->cAlts++]; as it will increment after the assignment
	911	unsigned cAlts = pIf->cAlts++;
	912	pIf = &paAlts[cAlts];
905	913	*pIf = If;
	914	fIfAdopted = true;
906	915	}
907	916	else
…	…
916	925	}
917	926	}
	927
	928	if (!fIfAdopted)
	929	freeInterfaceMembers (&If, 1);
918	930
919	931	/* start anew with endpoints. */

Note: See TracChangeset for help on using the changeset viewer.