Closed Bug 643137 Opened 9 years ago Closed 9 years ago
strict alignment issues when displaying Open
Type fonts on sparc64
I've been reported this issue, which probably affects 1.9.1, 1.9.2 and 2.0 as the ots code is the same for all branches. Firefox crashes when displaying http://thebrowsereview.com/category/css3/ on sparc64. The attached patch fixes the issue for the submitter, it should probably be pushed upstream (chromium ?) It's a followup of https://bugzilla.mozilla.org/show_bug.cgi?id=527276 where OTS landed in all branches.
Attachment #520448 - Flags: review?(roc) → review+
2.0 doesn't build/run on sparc64, but it'd be good if someone could land that in 1.9.1, 1.9.2 and 2.0 for the next point releases.
For reference, upstream bug report can be found here: https://code.google.com/p/chromium/issues/detail?id=76882
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Do i need to open another bug asking for a backport to 1.9.1, 1.9.2 & 2.0 ?
Comment on attachment 520448 [details] [diff] [review] Fix alignment issues on strict archs Setting approval flag for 1.9.1/1.9.2/2.0 to ? Please correct if it's not the right process...
Comment on attachment 520448 [details] [diff] [review] Fix alignment issues on strict archs The 1.9.1 branch didn't include OTS and we're not planning further releases off the 2.0 branch. If there are any they will be emergency security fixes for which this patch would be inappropriate. Leaving the 1.9.2 request open
Comment on attachment 520448 [details] [diff] [review] Fix alignment issues on strict archs Didn't make 3.6.18
Attachment #520448 - Flags: approval22.214.171.124+ → approval126.96.36.199-
And what's needed to get that commited ? approval-xxx is not enough now ?
The 2.0 branch is dead, removing checkin-needed.
Comment on attachment 520448 [details] [diff] [review] Fix alignment issues on strict archs Probably hopeless, but setting approval188.8.131.52 to ?
Attachment #520448 - Flags: approval184.108.40.206?
Comment on attachment 520448 [details] [diff] [review] Fix alignment issues on strict archs Approved for 220.127.116.11.
Attachment #520448 - Flags: approval18.104.22.168? → approval22.214.171.124+
Landry Breuil, can you verify this with the 1.9.2 candidates at ftp://ftp.mozilla.org/pub/firefox/nightly/3.6.20-candidates/. I have no Sparc64 machines with which to verify this fix.
Given that we(OpenBSD)'ve been shipping firefox 3.6.x with that patch since 3.6.16, i'd say it's been well enough tested. I have a limited time for that until 3.6.20 release..
3.6.20 Candidate build 1 works fine on sparc64 with that patch integrated, i can open http://thebrowsereview.com/category/css3/ without crashes. thanks!
Marking as verified for 1.9.2. Thanks!
You need to log in before you can comment on or make changes to this bug.