You were getting very lucky if you were not initializing your pointers to NULL. I'm surprised things weren't crashing immediately. <smile/>
On Wed, Mar 20, 2013 at 10:48 AM, Tristen Fielding < tfield...@tradestation.com> wrote: > Thanks. Sometimes the obvious thing is the thing you overlook. This code > had been working just fine for many years (under 3.5) and now it is failing > (under 3.7) all because we aren't initializing our pointers to null before > call WcaGetProperty. > > Thanks for the tip! > > Tristen Fielding > > -----Original Message----- > From: Hoover, Jacob [mailto:jacob.hoo...@greenheck.com] > Sent: Wednesday, March 20, 2013 1:04 PM > To: General discussion for Windows Installer XML toolset. > Cc: WiX-users@lists.sourceforge.net > Subject: Re: [WiX-users] WcaGetProperty is returning OutOfMemory Error... > > Just a guess but you aren't initializing your pointer to null. Possibly > the API is trying to free the old string before allocating a new one... > > > > On Mar 20, 2013, at 11:45 AM, "Tristen Fielding" > <tfield...@tradestation.com> wrote: > > > We have a custom action that was working just fine in 3.5, we are > upgrading to 3.7 and have noticed that for some reason we are getting an > Out of Memory errors anytime we call WcaGetProperty. > > > > Our first entry point is called BeginInstall and here are the first few > lines of code: > > > > > > UINT __stdcall BeginInstall(MSIHANDLE hModule) { > > CheckForDebugBreak(); > > > > HRESULT hr=::WcaInitialize(hModule, "BeginInstall"); > > ExitOnFailure(hr, "failed to initialize"); > > > > // Test code... > > LPWSTR installFolder; > > hr=::WcaGetProperty(L"INSTALLDIR", &installFolder); > > ExitOnFailure(hr, "failed to get INSTALLDIR property value"); > > ... > > > > Our current thinking is that somehow we are linking to the wrong version > of wcautil.lib. We are using VS2012 building against VS2010 tool set, but > we have tried both VS2012 and VS2010 both are failing with the same error > code. Any idea? > > > > > > Tristen Fielding > > Sr. Software Developer / Program Lead > > TradeStation Group, Inc. > > http://www.TradeStation.com<http://www.tradestation.com/> > > > > 8050 SW 10th Street, Suite 2000 > > Plantation, FL 33324 > > Tel. 954.652.7320 > > Fax 954.652.5320 > > > > > > > > TradeStation Group, Inc. is the parent company of four operating > subsidiaries, TradeStation Securities, Inc. (Member NYSE, FINRA, SIPC and > NFA), IBFX, Inc. (Member NFA), TradeStation Technologies, Inc., a trading > software and subscription company, and TradeStation Europe Limited, a > United Kingdom, FSA-authorized introducing brokerage firm. None of these > companies provides trading or investment advice, recommendations or > endorsements of any kind. The information transmitted is intended only for > the person or entity to which it is addressed and may contain confidential > and/or privileged material. Any review, retransmission, dissemination or > other use of, or taking of any action in reliance upon, this information by > persons or entities other than the intended recipient is prohibited. If you > received this in error, please contact the sender and delete the material > from any computer. Forex products and services are offered by TradeStation > Forex, a division of IBFX, Inc. > > > > ---------------------------------------------------------------------- > > -------- Everyone hates slow websites. So do we. > > Make your web apps faster with AppDynamics Download AppDynamics Lite > > for free today: > > http://p.sf.net/sfu/appdyn_d2d_mar > > _______________________________________________ > > WiX-users mailing list > > WiX-users@lists.sourceforge.net > > https://lists.sourceforge.net/lists/listinfo/wix-users > > > ------------------------------------------------------------------------------ > Everyone hates slow websites. So do we. > Make your web apps faster with AppDynamics Download AppDynamics Lite for > free today: > http://p.sf.net/sfu/appdyn_d2d_mar > _______________________________________________ > WiX-users mailing list > WiX-users@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/wix-users > > > TradeStation Group, Inc. is the parent company of four operating > subsidiaries, TradeStation Securities, Inc. (Member NYSE, FINRA, SIPC and > NFA), IBFX, Inc. (Member NFA), TradeStation Technologies, Inc., a trading > software and subscription company, and TradeStation Europe Limited, a > United Kingdom, FSA-authorized introducing brokerage firm. None of these > companies provides trading or investment advice, recommendations or > endorsements of any kind. The information transmitted is intended only for > the person or entity to which it is addressed and may contain confidential > and/or privileged material. Any review, retransmission, dissemination or > other use of, or taking of any action in reliance upon, this information by > persons or entities other than the intended recipient is prohibited. If you > received this in error, please contact the sender and delete the material > from any computer. Forex products and services are offered by TradeStation > Forex, a division of IBFX, Inc. > > > > > ------------------------------------------------------------------------------ > Everyone hates slow websites. So do we. > Make your web apps faster with AppDynamics > Download AppDynamics Lite for free today: > http://p.sf.net/sfu/appdyn_d2d_mar > _______________________________________________ > WiX-users mailing list > WiX-users@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/wix-users > > ------------------------------------------------------------------------------ Everyone hates slow websites. So do we. Make your web apps faster with AppDynamics Download AppDynamics Lite for free today: http://p.sf.net/sfu/appdyn_d2d_mar _______________________________________________ WiX-users mailing list WiX-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wix-users
