Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Performance Counter Manifest fails to register #4610

Closed
wixbot opened this issue Nov 25, 2014 · 4 comments
Closed

Performance Counter Manifest fails to register #4610

wixbot opened this issue Nov 25, 2014 · 4 comments
Assignees
Milestone

Comments

@wixbot
Copy link

wixbot commented Nov 25, 2014

The PerfCounterManifest extension is not registering the performance counter manifest
I have confirmed this running the PerfCounterManifest_install unit tests.

The bug is in the following code in scamanifest.cpp (40) ConfigurePerfmonManifestRegister(..)

line 105 is
hr = StrAllocFormatted(&pwzData, L""lodctr.exe" /m:"%s"", pwzFile);
should be the following
hr = StrAllocFormatted(&pwzCommand, L""lodctr.exe" /m:"%s"", pwzFile);

Originally opened by mstangl

@wixbot
Copy link
Author

wixbot commented Dec 4, 2014

Release changed from v3.9 to v3.10

@wixbot
Copy link
Author

wixbot commented Feb 1, 2015

Thanks for digging in. wixtoolset/wix3#196

Originally posted by barnson

@wixbot
Copy link
Author

wixbot commented Feb 1, 2015

Originally changed by barnson
AssignedTo set to bobarnson

@wixbot
Copy link
Author

wixbot commented Feb 4, 2015

e4679f6b38f5e44f8858236c48e5ead77fe47964

Originally posted by barnson
Resolution set to fixed
Status changed from Open to Resolved

@wixbot wixbot added this to the v3.10 milestone Dec 20, 2015
@wixbot wixbot closed this as completed Dec 20, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants