551153 - ###!!! ASSERTION: Not a UTF-8 string when running TestUTF and TestAtoms

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[Mark Banner (:standard8)]

Attachment: The fix

Following bug 534136, debug builds are showing the following assertion when running TestUTF:

###!!! ASSERTION: Not a UTF-8 string. This code should only be used for converting from known UTF-8 strings.: 'Error', file ../../dist/include/nsUTF8Utils.h, line 100
nsCRT::moz_strdup(unsigned short const*)+0x000003EC [/Users/moztest/comm/main/tb/mozilla/dist/bin/./libxpcom_core.dylib +0x00024A68]
nsCRT::HashCodeAsUTF16(char const*, unsigned int, int*)+0x00000043 [/Users/moztest/comm/main/tb/mozilla/dist/bin/./libxpcom_core.dylib +0x000243CF]

From what I can tell, this is a test to check that a non-UTF8 string is handled correctly, and therefore it shouldn't fail the test.

If I fix that, then TestAtoms also gets a similar assertion.

Therefore I think its right that we just disable the part of the tests that is giving the assertion for debug builds (obviously release builds will still run the test).

Comment 1

[Robert Kaiser]

Am I right that there are a few tests that SeaMonkey runs but Firefox doesn't due to being libxul? Is there a bug filed on that? It doesn't make too much sense if we have core tests that aren't run by Firefox...

Comment 2

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Comment on attachment 431345 [details] [diff] [review]
The fix

I'm actually fixing most of this in bug 551129 already. Though had missed that the hashas16 asserts. However we should only disable the part of the test that uses the Malformed8Strings array. Mind disabling just that in this patch?

Comment 3

[Serge Gautherie (:sgautherie)]

Attachment: (Bv1) Make it fully explicit that Malformed8Strings[] is not to be used in debug builds [Checkin: Comment 4]

Per comment 2.

NB: I assume Mark won't mind if I take over.

Comment 4

[Serge Gautherie (:sgautherie)]

Comment on attachment 431654 [details] [diff] [review]
(Bv1) Make it fully explicit that Malformed8Strings[] is not to be used in debug builds
[Checkin: Comment 4]


http://hg.mozilla.org/mozilla-central/rev/1912459c016b