Closed Bug 387726 Opened 17 years ago Closed 17 years ago

nsIXULAppInfo.platformVersion contains extra unknown characters

Categories

(Toolkit Graveyard :: Build Config, defect)

x86
All
defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED
mozilla1.9alpha8

People

(Reporter: mossop, Assigned: mossop)

References

Details

(Keywords: regression)

Attachments

(1 file)

Since fairly recently (I'd take a stab at bug 383167 but haven't verified) nsIXULAppInfo.platformVersion is now returning a 19 character string. This is "1.9a7pre" followed by 11 null characters. I've verified this on Windows and OSX, both on current Minefield nightlies.

Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.9a7pre) Gecko/2007071004 Minefield/3.0a7pre ID:2007071004
Attached patch patch rev 1Splinter Review
Apparently AssignLiteral shouldn't be used with a character array. This resolves the issue.
Assignee: nobody → dtownsend
Status: NEW → ASSIGNED
Attachment #272944 - Flags: review?(benjamin)
Blocks: 383167
Comment on attachment 272944 [details] [diff] [review]
patch rev 1

Whoops, you're right.
Attachment #272944 - Flags: review?(benjamin) → review+
Checking in toolkit/xre/nsAppRunner.cpp;
/cvsroot/mozilla/toolkit/xre/nsAppRunner.cpp,v  <--  nsAppRunner.cpp
new revision: 1.182; previous revision: 1.181
done
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Flags: in-testsuite?
Target Milestone: --- → mozilla1.9beta1
Product: Toolkit → Toolkit Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: