Bug 43844 - wocky-connector-test crashes in windows
Summary: wocky-connector-test crashes in windows
Status: RESOLVED FIXED
Alias: None
Product: Wocky
Classification: Unclassified
Component: General (show other bugs)
Version: git master
Hardware: x86 (IA32) Windows (All)
: medium normal
Assignee: Siraj Razick
QA Contact: Telepathy bugs list
URL: http://cgit.collabora.com/git/user/si...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-12-14 17:14 UTC by Siraj Razick
Modified: 2012-04-05 08:12 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
wocky log : windows (8.35 KB, text/plain)
2011-12-16 13:11 UTC, Siraj Razick
Details

Description Siraj Razick 2011-12-14 17:14:13 UTC
This test case works on windows with wine, but fails on windows 7. and the library is cross compiled on Linux with mingw32

C:\Users\siraj\gllib\windows7\root\bin>wocky-connector-test.exe
wocky-connector-test-Message: libsasl2 not found: skipping MD5 SASL tests
/connector/basic/internals: Error: wocky-connector-error.7: TLS handshake erro
-9: GNUTLS_E_UNEXPECTED_PACKET_LENGTH **
wocky-connector-test:ERROR:wocky-connector-test.c:3396:run_test: assertion fai
d (error == NULL): TLS handshake error: -9: GNUTLS_E_UNEXPECTED_PACKET_LENGTH
ocky-connector-error, 7)

This application has requested the Runtime to terminate it in an unusual way.
Please contact the application's support team for more information.

C:\Users\siraj\gllib\windows7\root\bin>
Comment 1 Siraj Razick 2011-12-16 13:11:31 UTC
Created attachment 54517 [details]
wocky log : windows

wocky debug log
Comment 2 Siraj Razick 2011-12-16 14:19:04 UTC
from the log it was so obvious that the error was due the paths of the cert files,
it was hard coded during the cross compile so it never worked on windows.
so i guess the problem is fix.. need to find a clean way to solve this.

maybe using relative path when running it ? so that certs/ directory is picked up from current path the test-case? (only for windows) or when cross compiling is detect .
Comment 3 Siraj Razick 2012-02-01 10:23:05 UTC
patch
Comment 4 Jonny Lamb 2012-02-01 10:28:54 UTC
Is this a fix for the path problem? I don't understand.
Comment 5 Siraj Razick 2012-02-01 10:33:10 UTC
(In reply to comment #4)
> Is this a fix for the path problem? I don't understand.

stefw told me to add an assert to make the problem more obvious by adding some asserts I can't remember why we discussed it else were and not here.

<<
Updated by Stef Walter about 1 month ago
Comment Edit

Did you send a patch upstream to give a better error message in this corner case? We discussed doing this on IRC, so that the next person who comes along doesn't have to figure out the same thing.

>>
Comment 6 Nicolas Dufresne 2012-04-05 07:36:26 UTC
> add_certfiles: wocky-tls.c:545: checking /home\siraj\projects\collabora\windows\git\tp-windows\wocky\tests\certs\ca-0-cert.pem

Looks like the certificate path are all wrong.
Comment 7 Nicolas Dufresne 2012-04-05 07:39:34 UTC
Ok, didn't see the link. Basically you didn't properly set the path at build time and it fail, I agree this patch should make it more obvious.

++
Comment 8 Nicolas Dufresne 2012-04-05 07:50:40 UTC
Correction, this fails the tests for me.
Comment 9 Nicolas Dufresne 2012-04-05 08:12:25 UTC
Ok, this is because I removed the CRL, it builds because I forgot to remove the -DTLS_CRL_DIR in the tests Makefile.am.

Fixed and merged.


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.