(These have proved invaluable for other global accessors.)

vagrant authored 11 years ago and Nick Mathewson committed 11 years ago

Change the global circ_times to a static variable and use
accessor functions throughout the code, instead of
accessing it directly.

Peter Retzlaff authored 11 years ago and Nick Mathewson committed 11 years ago

Implements ticket 5526.

Conflicts:
	src/or/circuitlist.h

George Kadianakis authored 11 years ago and Nick Mathewson committed 11 years ago

- Set conn->address when we receive a USERADDR command.
- Set conn->state to a sane value when we transition from Extended
  ORPort to ORPort.

George Kadianakis authored 11 years ago and Nick Mathewson committed 11 years ago

We started allocating space for them on the heap in the previous
commit.

Conflicts:
	src/or/ext_orport.h

George Kadianakis authored 11 years ago and Nick Mathewson committed 11 years ago

Use the generic function for both the ControlPort cookie and the
ExtORPort cookie.

Also, place the global cookie variables in the heap so that we can
pass them around more easily as pointers.

Also also, fix the unit tests that broke by this change.

Conflicts:
	src/or/config.h
	src/or/ext_orport.c

(Okay, white-box plus mocking enough other functions so they don't
crash.)

Incidentally, this business here where I make crypto_rand mockable:
this is exactly the kind of thing that would make me never want to
include test-support stuff in production builds.

No other changes were made here.  Keeping everything in
src/test/test.c was a legacy of back when we had all our unit tests in
one big file.

Doing this now because I'm adding an ext_or_command test.

Remove a nedless strdup/free pair.

memwipe some stack-allocated stuff
Add DOCDOC comments for state machines
Use memdup_nulterm as appropriate
Check for NULs in useraddr
Add a macro so that <= AUTH_MAX has a meaning.