Summary: | Possible exception/segfault in jurt jpipe.dll under Windows with pipe connection | ||
---|---|---|---|
Product: | LibreOffice | Reporter: | Juergen Funk <juergen.funk_ml> |
Component: | sdk | Assignee: | Juergen Funk <juergen.funk_ml> |
Status: | ASSIGNED --- | QA Contact: | |
Severity: | normal | ||
Priority: | medium | CC: | sbergman, serval2412 |
Version: | 4.5.0.0.alpha0+ Master | ||
Hardware: | x86 (IA32) | ||
OS: | Windows (All) | ||
Whiteboard: | target:4.5.0 | ||
i915 platform: | i915 features: | ||
Attachments: | Test this exception with this JAVA Class |
Description
Juergen Funk
2014-11-26 15:34:18 UTC
Here's some code pointer I suppose: http://opengrok.libreoffice.org/xref/core/jurt/source/pipe/wrapper/wrapper.c#44 Stephan: one for you? Yes, I faintly remember that the DllMain restrictions were well known when the code was originally written, but it was considered "appears to work anyway." But should be possible to move the code from DllMain into getFunction. Jürgen, do you want to work on that? - Remove the LoadLibrary from DLLMain (from windows not recommended) - Set function-ptr variable (faster) - Improve the comment why we need two dll's (jpipe.dll and jpipx.dll) gerrit: https://gerrit.libreoffice.org/#/c/13148/ In Patch set 2 i have remove the function pointer gerrit: see Comment 2 Juergen Funk committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=b21f2439ad2681f906c63c2192ef141d9abb9e37 fdo#86745 - Possible exception/segfault in jurt jpipe.dll under Windows ... It will be available in 4.5.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.