Project

General

Profile

Actions
Copy link

Bug #4476

closed

Wrong ValidityPeriod.NotAfter in certificate expiring after year 2038

Added by Junxiao Shi almost 7 years ago. Updated almost 7 years ago.

Status:
Closed
Priority:
Urgent
Assignee:
Alex Afanasyev
Category:
Security
Target version:
v0.7
Start date:
01/25/2018
Due date:
% Done:

100%

Estimated time:

Description

In test case ndn-cxx.Tests.Security.V2.TestKeyChain.SelfSignedCertValidity:

[Error] - check cert.isValid() failed
 == [File] - ../tests/unit-tests/security/v2/key-chain.t.cpp
 == [Line] - 398

[Error] - check cert.isValid(time::system_clock::now() + 10 * 365_days) failed
 == [File] - ../tests/unit-tests/security/v2/key-chain.t.cpp
 == [Line] - 399

[Error] - check cert.getValidityPeriod().getPeriod().second > time::system_clock::now() + 10 * 365_days failed [-2147401558000000000 nanoseconds since Jan 1, 1970 <= 1832205738150935238 nanoseconds since Jan 1, 1970]
 == [File] - ../tests/unit-tests/security/v2/key-chain.t.cpp
 == [Line] - 400

Related issues 2 (0 open — 2 closed)

Related to NLSR - Bug #4473: Running into year 2038 problem as certificate's validity period is hard coded to 20 yearsClosedAshlesh Gawande01/24/2018

Actions
Related to ndn-cxx - Bug #4478: time::fromIsoString doesn't decode large datesClosedAlex Afanasyev01/25/2018

Actions
Actions
Copy link
 #1

Updated by Junxiao Shi almost 7 years ago

  • Related to Bug #4473: Running into year 2038 problem as certificate's validity period is hard coded to 20 years added
Actions
Copy link
 #2

Updated by Alex Afanasyev almost 7 years ago

  • Related to Bug #4478: time::fromIsoString doesn't decode large dates added
Actions
Copy link
 #3

Updated by Junxiao Shi almost 7 years ago

  • Related to deleted (Bug #4478: time::fromIsoString doesn't decode large dates)
Actions
Copy link
 #4

Updated by Junxiao Shi almost 7 years ago

  • Is duplicate of Bug #4478: time::fromIsoString doesn't decode large dates added
Actions
Copy link
 #5

Updated by Alex Afanasyev almost 7 years ago

  • Is duplicate of deleted (Bug #4478: time::fromIsoString doesn't decode large dates)
Actions
Copy link
 #6

Updated by Alex Afanasyev almost 7 years ago

  • Related to Bug #4478: time::fromIsoString doesn't decode large dates added
Actions
Copy link
 #7

Updated by Alex Afanasyev almost 7 years ago

Issue #4478 is not duplicate. It is the underlying cause. This issue describes the high-level problem.

The relation, as commented in #4478, is because ValidityPeriod encodes/decodes dates to/from ISO format.

Actions
Copy link
 #8

Updated by Davide Pesavento almost 7 years ago

  • Status changed from New to Closed
  • Assignee set to Alex Afanasyev
  • % Done changed from 0 to 100

Should be fixed now.

Actions
Copy link

Also available in: Atom PDF