Openrave crashes reading Collada zae on Windows 7 due to invalid URI in daeIOPluginCommon.cpp about collada-dom HOT 4 CLOSED

Proposed fix is to convert the rootFilePathto a proper URI before passing it to the daeURI constructor:

diff --git a/dom/src/dae/daeIOPluginCommon.cpp b/dom/src/dae/daeIOPluginCommon.cpp
index 1553454..b18555f 100644
--- a/dom/src/dae/daeIOPluginCommon.cpp
+++ b/dom/src/dae/daeIOPluginCommon.cpp
@@ -92,7 +92,7 @@ daeInt daeIOPluginCommon::read(const daeURI& uri, daeString docBuffer)
         if (zaeHandler.isZipFile())
         {
             string rootFilePath = zaeHandler.obtainRootFilePath();
-            daeURI rootFileURI(*fileURI.getDAE(), rootFilePath);
+           daeURI rootFileURI(*fileURI.getDAE(),cdom::nativePathToUri(rootFilePath));
             domObject = readFromFile(rootFileURI);
             if (!domObject)
             {

After this fix, the tutorial_ik5d example (and others) work as expected

from collada-dom.

it must have been an adventure getting windows to work. thanks for the patch ;0)

from collada-dom.

Hi Rosen.
Thank you.
Please let me know if you are interested in the patches I had to make to the OpenRave master as well. I compiled with Boost 1.46.1, using Python 2.6 and had to do quite a few changes to the Openrave libraries in order to make it compile and run.

Regards,
Per

From: Rosen Diankov [mailto:[email protected]]
Sent: 8. september 2014 15:07
To: rdiankov/collada-dom
Cc: Per Sandved Hustad
Subject: Re: [collada-dom] Openrave crashes reading Collada zae on Windows 7 due to invalid URI in daeIOPluginCommon.cpp (#8)

it must have been an adventure getting windows to work. thanks for the patch ;0)

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/8#issuecomment-54814224.

from collada-dom.

sure! please send them.

from collada-dom.