formattm test crashes in strftime call on Android

RESOLVED FIXED

Status

RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: bcombee, Assigned: alexp)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

9 years ago
The formattm test crashes when run on Android 2.0.1.  This looks to be a bug in the bionic libc, but I've not yet tracked down which format string item is the problem.  Through tracing in the debugger, I was able to determine the crash was inside the strftime code and that the C library had been linked correctly.
(Assignee)

Updated

9 years ago
Assignee: wtc → alexp
(Assignee)

Updated

9 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 1

9 years ago
Created attachment 435312 [details] [diff] [review]
Fix

struct tm on Android has additional fields, which were not initialized in PR_FormatTime function (prtime.c).
Attachment #435312 - Flags: superreview?(wtc)
Attachment #435312 - Flags: review?(bugmail)
Attachment #435312 - Flags: review?(bugmail) → review+

Updated

9 years ago
Depends on: 542113
Attachment #435312 - Flags: superreview?(wtc) → superreview+
Landed in CVS:
Checking in pr/src/misc/prtime.c;
/cvsroot/mozilla/nsprpub/pr/src/misc/prtime.c,v  <--  prtime.c
new revision: 3.44; previous revision: 3.43
done
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.