Tobias Schäfer wrote:
> Hi devs,
>
> In the following independant scenarios TortoiseProc crashes when starting.
>
> Scenario 1:
> Place versions of the files CrashRpt.dll and zlib.dll which are not
> compatible to TSVN into a directory and add this directory to %path%. Any
> call to TortoiseProc will crash the application.
>
> Scenario 2:
> Place versions of the files CrashRpt.dll and zlib.dll which are not
> compatible to TSVN into any Subversion controlled directory and try to "SVN
> Add" them.
>
> Incompatible versions of the two files can be found here:
> http://www.codeproject.com/debug/crash_report/crashrpt_demo.zip
>
> In both cases TSVN crashes because the constructor of CCrashReport
> successfully loads CrashRpt.dll ("LoadLibrary") but then fails to find the
> function InstallEx ("GetProcAddress")
>
> A simple patch is attached. This prevents TSVN from crashing on startup but
> crash reporting is obviously not available then.
>
> I would like to commit this and merge it into the 1.4.x branch. Stefan, is
> that ok with you or do want a superior solution? Idea: We could pass a
> full absolute path to "LoadLibrary" so that TSVN gets the correct version.
> I'm surprised to not find CrashRpt.dll in the bin directory of the TSVN
> installation. Where is it?
Your patch is fine by me. Go ahead with the commit.
For later, we really should find the absolute path and then load the dll
only from our own installation folder. But that should wait (i.e. on
trunk, not to be merged back to 1.4.x).
The crashrpt.dll is only installed with official releases where I keep
the debug symbols. It's not installed in nightly builds.
Stefan
--
___
oo // \\ "De Chelonian Mobile"
(_,\/ \_/ \ TortoiseSVN
\ \_/_\_/> The coolest Interface to (Sub)Version Control
/_/ \_\ http://tortoisesvn.net
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org
Received on Mon Aug 21 19:01:15 2006