70254 - radio buttons w/ type string crash in SetUnicharPref()

Status: VERIFIED INVALID

(SeaMonkey :: Preferences, defect)

Description

[Chris McAfee]

From bug 68208, I had to change pref type from string
to int to avoid crashing deep in layout somewhere
(sorry, linux/gdb failed me for libgkcontent.so).

Comment 1

[Chris McAfee]

Adding bstell for i18n look.

Comment 2

[kill this account]

At present I have not researched why changing from string to int
fixed bug 68208 but the fix/description makes me very uncomfortable.

Based on the fix it would appear that one part of the code assumes
the data type is string and another part assumes it is int.

Did we understand how confusion between the two parts occurred
before we checked in the fix?

Did we check to see if this problem is not happening in other
areas?

Comment 3

[Chris McAfee]

I didn't have time to investigate the problem with the string type,
hence this bug.  I got the other bug off of the nsbeta1 radar,
that radio pref should have been an int anyways.

Comment 4

[Chris McAfee]

my fault, I incorrectly pointed my finger at xul/radios.
bogus fixed backed out, marking this invalid.

Comment 5

[sairuh (rarely reading bugmail)]

zap