was previously replaced with a NUL for parsing purposes.  This seems to
fix a very weird parsing bug involving two variable references in the same
value.

with a read lock

Submitted by: Leandro Santi <lesanti@fiuba7504.com.ar>

PR: 1196
Submitted by: Russel Ruby

PR: 1205

PR: 1196
Submitted by: Russel Ruby

PR: 1198

PR: 1198

Submitted by: Old Wolf, Jon Bright

PR: 1176 [and others]

PR: 1195

PR: 1201

Submitted by: Jonathon Green

PR: 1194

-Wmissing-prototypes -Wcomment -Wformat -Wimplicit -Wmain -Wmultichar
-Wswitch -Wshadow -Wtrigraphs -Werror -Wchar-subscripts
-Wstrict-prototypes -Wreturn-type -Wpointer-arith  -W -Wunused
-Wno-unused-parameter -Wuninitialized

runtime, thus removing the need for a lock. Add a test to ssltest
to verify that the cipher lists are sorted.

let's be clear about that

Submitted by: David Heine <dlheine@suif.Stanford.EDU>

PR: 1183