Closed Bug 450991 Opened 16 years ago Closed 16 years ago

Number in "Order Received" column becomes negative, when mail's offset in local mail folder file exceeds 2GB

Categories

(MailNews Core :: Backend, defect)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 3.0b1

People

(Reporter: World, Assigned: hiro)

References

Details

(Keywords: fixed1.8.1.18)

Attachments

(3 files)

> [App] Name=Thunderbird, Version=3.0b1pre, BuildID=20080814031842
>       ID={3550f703-e582-4d05-9a08-453d09bdfdc6}
> [Gecko] MinVersion=1.9.1a2pre, MaxVersion=1.9.1a2pre
> On MS Win-XP SP3

Number in "Order Received" column becomes negative, when mail's offset in local mail folder file exceeds 2GB.
Sort order is proper(sorted by value of 32bit unsigned integer). So it looks to be display only issue(treated as 32bit signed integer). Possibly due to common module for Win,Linux,Mac OS. (Mail folder file greater then 2GB is supported by NSPR for Win only)

Note:
This bug's problem is minor. But critical issues such as Bug 450359 really exist, due to insuffient changes after support of mail folder file greater than 2GB.
support 4GB over mailbox.
Attachment #334367 - Flags: review?(bienvenu)
Attachment #334372 - Flags: review?(bienvenu)
Comment on attachment 334372 [details] [diff] [review]
Same patch for trunk
[Checkin: Comment 5]

thx for the patch
Attachment #334372 - Flags: review?(bienvenu) → review+
Comment on attachment 334367 [details] [diff] [review]
Workaround patch for 1.8 branch

thx for the patch
Attachment #334367 - Flags: superreview?(neil)
Attachment #334367 - Flags: review?(bienvenu)
Attachment #334367 - Flags: review+
Attachment #334372 - Flags: superreview?(neil)
Attachment #334367 - Flags: superreview?(neil) → superreview+
Attachment #334372 - Flags: superreview?(neil) → superreview+
Assignee: nobody → poincare
Status: NEW → ASSIGNED
Component: Mail Window Front End → Backend
Keywords: checkin-needed
OS: Windows XP → All
Product: Thunderbird → MailNews Core
QA Contact: front-end → backend
Hardware: PC → All
Target Milestone: --- → Thunderbird 3.0b1
Whiteboard: [Why 2 patches.?.]
Attachment #334367 - Attachment description: Workaround patch → Workaround patch for 1.8 branch
Whiteboard: [Why 2 patches.?.] → [Th first one is for 1.8 branch, another one is for trunk]
Comment on attachment 334372 [details] [diff] [review]
Same patch for trunk
[Checkin: Comment 5]

http://hg.mozilla.org/comm-central/rev/bab601fb4812
Attachment #334372 - Attachment description: Same patch for trunk. → Same patch for trunk [Checkin: Comment 5]
I think you need to get some kind of additional approval before branch checkin.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Whiteboard: [Th first one is for 1.8 branch, another one is for trunk] → [c-n: 1.8.1 branch]
Attachment #334367 - Flags: approval1.8.1.18?
I don't know if it is related, but I cannot unfold any thread in mailnews or click on the top list message without getting a crash.
Please add checkin-needed back once this has got approval.

Someone also needs to confirm whether or not comment 7 is a regression from this.
Keywords: checkin-needed
Frederic: probably best to file a followup bug. But anyway, can you give the breakpad crash id? Is the folder in question over 2GB?
Comment on attachment 334367 [details] [diff] [review]
Workaround patch for 1.8 branch

a=dmose; however, bugzilla doesn't seem to think I have authority to set approval1.8.1.18+ for some reason.
Attachment #334367 - Flags: approval1.8.1.18? → approval1.8.1.18+
Magnus : I target wrong bug as guilty for my crashes. It was bug 460058 here. Sorry for my mistake :/
fix checked into the 1.8 branch
Keywords: fixed1.8.1.18
Whiteboard: [c-n: 1.8.1 branch]
You need to log in before you can comment on or make changes to this bug.