test 618cgisecurity.t: Remove semi-colons from test

We still don't match the "unlike", so I have no idea what extra
protections the semicolons add.

Signed-off-by: Andreas Ericsson <ae@op5.se>
test: Remove ridonculous tests

The 620history.t and 621 extinfo.t tests only provide a html lint
scan, but relies on Test::WWW::Mechanize::CGI, which no current
distribution ships by default. Since that means it's almost certainly
too much hassle for package developers to bother with, we get rid
of them completely so we have a chance of getting people (and me) to
run 'make test' on a regular basis, since it does catch actual
errors from time to time and marks incompatibilities when they're
introduced.

Signed-off-by: Andreas Ericsson <ae@op5.se>
lib/kvvec.h: Add missing declaration for kvvec_free_kvpairs()

It's a public function according to its definition, so make sure
it's usable from elsewhere too.

Signed-off-by: Andreas Ericsson <ae@op5.se>
cgi: Janitor patch. Remove set-but-unused variables

We don't need them, so byebye.

Signed-off-by: Andreas Ericsson <ae@op5.se>
Let cgi's use object pointers too

It's costly and inefficient to use two separate object layouts
depending on whether we're building for cgi's or the daemon, and
especially so when the separate object layouts means we require a
lot of extra code to provide the same functionality to both parts,
except one codepath is slower than the other.

With this patch we streamline the objects and let the cgi's use
object pointers the same way the core does when looking up members
and whatnot.

Signed-off-by: Andreas Ericsson <ae@op5.se>