Project

General

Profile

Actions

Bug #4476

closed

Wrong ValidityPeriod.NotAfter in certificate expiring after year 2038

Added by Junxiao Shi about 6 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Urgent
Category:
Security
Target version:
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 open2 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 #1

Updated by Junxiao Shi about 6 years ago

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

Updated by Alex Afanasyev about 6 years ago

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

Updated by Junxiao Shi about 6 years ago

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

Updated by Junxiao Shi about 6 years ago

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

Updated by Alex Afanasyev about 6 years ago

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

Updated by Alex Afanasyev about 6 years ago

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

Updated by Alex Afanasyev about 6 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 #8

Updated by Davide Pesavento about 6 years ago

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

Should be fixed now.

Actions

Also available in: Atom PDF