If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Potential Y2k problem at nsFTPDirListingConv.cpp:778

VERIFIED FIXED in M12

Status

SeaMonkey
General
P3
normal
VERIFIED FIXED
18 years ago
13 years ago

People

(Reporter: Anup Parkhi, Assigned: Judson Valeski)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

18 years ago
Please look at the following code at
./mozilla/netwerk/streamconv/converters/nsFTPDirListingConv.cpp:778

curTime.tm_year     = (((aCStr[6]-'0')*10) + aCStr[7]-'0');

The curTime is of type PRExplodedTime. The tm_year in PRExplodedTime represent
absolute year (four digit). It appears to me that RHS of the above statement
will result in two digit year only. Hence the curTime.tm_year will get assigned
the two digit year which i think does not match the defintion of tm_year in
PRExplodedTime.

Updated

18 years ago
Assignee: leger → valeski
QA Contact: anupp → shrir

Comment 1

18 years ago
Assigning to valeski
(Assignee)

Updated

18 years ago
Status: NEW → ASSIGNED
Target Milestone: M12
(Assignee)

Comment 2

18 years ago
I've got a fix. atatching...
(Assignee)

Comment 3

18 years ago
Created attachment 2591 [details] [diff] [review]
patch to fix y2k prob in dir listing conv
(Assignee)

Updated

18 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
(Assignee)

Comment 4

18 years ago
fix checked in 11/10/99 4:30pm pac time.

Updated

18 years ago
Status: RESOLVED → VERIFIED

Comment 5

18 years ago
Marking Verified/Fixed.

Comment 6

17 years ago
moving to browser-general component.  the `Y2k' component is being retired...
Component: Y2k → Browser-General
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.