Skip to content
This repository has been archived by the owner on Jun 28, 2022. It is now read-only.

RUSTSEC-2020-0071: Potential segfault in the time crate #28

Closed
github-actions bot opened this issue Oct 18, 2021 · 1 comment
Closed

RUSTSEC-2020-0071: Potential segfault in the time crate #28

github-actions bot opened this issue Oct 18, 2021 · 1 comment
Assignees

Comments

@github-actions
Copy link

Potential segfault in the time crate

Details
Package time
Version 0.1.44
URL time-rs/time#293
Date 2020-11-18
Patched versions >=0.2.23
Unaffected versions =0.2.0,=0.2.1,=0.2.2,=0.2.3,=0.2.4,=0.2.5,=0.2.6

Impact

Unix-like operating systems may segfault due to dereferencing a dangling pointer in specific circumstances. This requires an environment variable to be set in a different thread than the affected functions. This may occur without the user's knowledge, notably in a third-party library.

The affected functions from time 0.2.7 through 0.2.22 are:

  • time::UtcOffset::local_offset_at
  • time::UtcOffset::try_local_offset_at
  • time::UtcOffset::current_local_offset
  • time::UtcOffset::try_current_local_offset
  • time::OffsetDateTime::now_local
  • time::OffsetDateTime::try_now_local

The affected functions in time 0.1 (all versions) are:

  • at
  • at_utc

Non-Unix targets (including Windows and wasm) are unaffected.

Patches

Pending a proper fix, the internal method that determines the local offset has been modified to always return None on the affected operating systems. This has the effect of returning an Err on the try_* methods and UTC on the non-try_* methods.

Users and library authors with time in their dependency tree should perform cargo update, which will pull in the updated, unaffected code.

Users of time 0.1 do not have a patch and should upgrade to an unaffected version: time 0.2.23 or greater or the 0.3. series.

Workarounds

No workarounds are known.

References

time-rs/time#293

See advisory page for additional details.

@SanchithHegde
Copy link
Owner

The dependents on the time crate as of today are:

time v0.1.43
└── chrono v0.4.13
    └── simplelog v0.10.2

We have a few choices to address this issue that I can think of:

  1. Wait till Bump time to 0.3 chronotope/chrono#578 is merged.
  2. Wait till simplelog replaces chrono with time in their dependency tree (may or may not happen).
  3. Rewrite the logging setup to replace simplelog with tracing and friends, or env_logger.

I will wait for a few days before going with option 3.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant