Opened 4 weeks ago

Last modified 4 weeks ago

#21356 merge_ready defect

[0.2.9.9] Bug: compat_time.c: monotime_coarse_absolute_nsec() called before monotime_init() on windows service

Reported by: cypherpunks Owned by: nickm
Priority: Medium Milestone: Tor: 0.2.9.x-final
Component: Core Tor/Tor Version:
Severity: Minor Keywords: windows 029-backport
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description (last modified by dgoulet)

[notice] Bootstrapped 85%: Finishing handshake with first hop
[warn] tor_bug_occurred_(): Bug: compat_time.c:636: monotime_coarse_absolute_nsec: Non-fatal assertion !(monotime_initialized == 0) failed. (on Tor 0.2.9.9 )
[warn] Bug: Non-fatal assertion !(monotime_initialized == 0) failed in monotime_coarse_absolute_nsec at compat_time.c:636. (Stack trace not available) (on Tor 0.2.9.9 )
[notice] Bootstrapped 90%: Establishing a Tor circuit

type: Win32 Tor service
result: show error only. Can connect to internet without problem.

Child Tickets

Change History (6)

comment:1 Changed 4 weeks ago by dgoulet

  • Description modified (diff)
  • Keywords windows added
  • Milestone set to Tor: 0.3.0.x-final
  • Type changed from task to defect

comment:2 Changed 4 weeks ago by nickm

  • Keywords 029-backport added
  • Severity changed from Normal to Minor
  • Summary changed from [0.2.9.9] Bug: compat_time.c to [0.2.9.9] Bug: compat_time.c: monotime_coarse_absolute_nsec() called before monotime_init() on windows service

I bet that this being a service is something to do with this.

(This bug is not actually harmful, but we should fix it anyway).

comment:3 Changed 4 weeks ago by nickm

  • Owner set to nickm
  • Status changed from new to accepted

bug21356_029 should be a straightforward fix here.

comment:4 Changed 4 weeks ago by nickm

  • Status changed from accepted to needs_review

comment:5 Changed 4 weeks ago by teor

  • Status changed from needs_review to merge_ready

It seems correct to me, and it's good that it's after logging.
Let's bake it in master for a while before a backport?

comment:6 Changed 4 weeks ago by nickm

  • Milestone changed from Tor: 0.3.0.x-final to Tor: 0.2.9.x-final

ok, merged!

Note: See TracTickets for help on using tickets.