Fix potential use of old connections after config change

Despite detecting and tagging them for close on config change,
the closing happened only in long-term maintenance.  So there
was time where they could still be reused by clients.

Fix this by strict check in find_server()

Fixed adjustment of a WSAStartup position problem.
It happen the hostname problem in the service start.
Per report Sachin Srivastava. and, thanks suggestions of Marko.

Fixed the Service parameter path name problem.
Per report from Sachin Srivastava.

Fix broken alignment in src/slab.c

Alignment macro was buggy, thus resulting in bad alignment.
This means speed hit on x86, but may result in crashes
on arches which are strict on alignment, unless the macro
is pre-defined in OS headers.

No problems from the field have been reported, thus maybe
even the bad alignment happened to be aligned?

SHOW DATABASES had wrong column count.

The new res_pool field was not mentioned in data rows.

update docs and default config

release 1.3

win32: improve service description

Hiroshi Saito

Apply ISC license clarification, bump copyright years.

Clarification:

  for f in */*.[ch]; do echo -e "%s;and distribute;and/or distribute;\nwq" | ed "$f"; done

Details at: http://en.wikipedia.org/wiki/ISC_licence

duh, seems we did not include COPYRIGHT file in tgz

README: mention reserve_pool feature

add missing win32 files to tgz

event_loopbreak compat for libevent 1.3

tag 1.3rc1

README: win32 info

doc/Makefile: build also html for README

NEWS: expand autodb and win32 items

win32: make service_name alias for job_name

win32 todo

NEWS: mention event_del problem

add few overlooked contributors

sbuf_close: dont retry event_del() if it fails, just proceed with cleanup.

The retry could have been bug in pgbouncer - although then the
question is that how to survive ENOMEM from event handlers that
may need allocation on event_del().

But this does not explain why it fails first time.  Seems there
is something funny going on in libevent.

win32: simpler service handling, cleanups

- one pgbouncer instance per service, in-process
- read service name from config file

cleanups:

- use FormatMessage for strerror()
- error out on wide-char build
- tag stdout/err _IONBF, they seem fully buffered by default

some more cleanups:

- avoid NULL values for config variables
- cf_shutdown=2 means fast shutdown
- export cf_config_file

minor cleanups

- avoid use of non-const fmt string
- avoid duplicate def of sbuf_len

draft news for 1.3

Reserve pool implementation.

Allow to increased pool size in case main pool size
cannot serve clients anymore.

This allows to continue use of small pool_size, which
smooths small peaks, while also handling large peaks.

(Eg, one bouncer server down, others need to handle load.)

check if connection was accepted before logging

win32: make asynctest compile again

By Hiroshi Saito

win32: add -A to def generation

win32: no need for -subservice switch, instead use -q

dlltool seems to need --export-all-symbols to make .def file

pgbevent: make g_module static var

win32: more fixes

- proper message id
- fix bugs in cmdline validation

locatime weirdness wasnt windows specific...

win32: tm_mon fix for localtime() (Hiroshi Saito)

win32: rename support files

win32: more cleanups

- emulate fcntl instead using #ifdef
- getsockopt() and select() also need wrapping
- move syslog declarations into compat_win32.h

win32: implement kill()

Patch by Hiroshi Saito, applied with some modifications.

win32: implement sendmsg, move cmdline handling away from main.c

win32: fix locatime

win32: errno fixes

- switch errors around to correspond to what gets returned
- somewhat working strerror()

win32: more fixes

- move socket errors back to errno
- don't call closesocket() on plain files

win32: change name of eventmsg

remove the mc.exe detection, it not available anyway

Quick win32 fixes.

It not only compiles now, but even boots..

remove unnecessary CVS ids, update AUTHORS

win32 support (wip)

Original patch by Hiroshi Saito, applied with some cleanup

Make crypt authentication optional.

It was removed from PostgreSQL 8.4 sources.

Dynamic database creation for non-defined databases.

Activated by "*" entry in [databases] section.

Patch by David Galoyan

asynctest improvements

- add $(LIBS) to link line
- allow custom query
- define fatal_noexit() for CASSERT builds

add milliseconds to log timestamps

New tunable 'sbuf_loopcnt' to limit time spent on one socket.

In some situations - eg SMP server, local Postgres and fast network -
pgbouncer can run recv()->send() loop many times without blocking
on either side.  But that means other connections will stall for
a long time.  To make processing more fair, limit the times
of doing recv()->send() one socket.  If count reaches limit,
just proceed processing other sockets.  The processing for
that socket will resume on next event loop.

Thanks to Alexander Schöcke for report and testing.

debug: log connection endpoints

Change default port number from 6000 to 6432.

6000 is conflicts with X, 6432 seems free and is already used by Debian.

quiet compiler warning about ptr->int conversion

stop playing with -fomit-frame-pointer

Default to --enable-debug

To get the old behaviour - stripped binary and -fomit-frame-pointer
in CFLAGS, --disable-debug must be used.

Allowing debugging by default seems saner.

Put own includes at the startt of CPPFLAGS

Otherwise with --with-libevent=/usr wrong includes may be used.

Per report from Bjorn Munch

compact md5 implementation

full quoting for database names

tag head as 1.3

include userlist.txt in tgz

disable untested SO_ACCEPTFILTER code

v1.2.3

detect stale pidfile

add Jørgen

allow first libevent test pass on 1.1

glibc requires use of _GNU_SOURCE

Change the makefile to use "$(MAKE)" instead of "make".

Patch from Jørgen Austvik

asynctest: remove note about splice() experiments

tag 1.2.2

Remove drop_on_error, it was a bad idea.

It was a workaround for broken plan-cache in Postgres,
but it's behviour for common case - some queries giving always errors -
is very nasty - it can drop all connections in pool.

asynctest: check if result comes back ok

mention log msg cleanup

v1.2.1

wording cleanup for drop_on_error

cancel shutdown on resume

otherwise admin gets bad surprise on next pause

exit immediately on SIGINT if suspend was in progress

suspend_socket_list can drop sockets, so needs _safe

proper log message for console client cancel

cleaner socket_row()

sys/uio.h is required for struct iover on some systems

(eg. DragonFly)

log if drop_on_error triggers

admin.c: dont crash if socket has no buffer

change ver to 1.2.1

new drop_on_error param

release 1.2

test/Makefile: add DEFS to cflags

tag 1.2rc2

test: increase allowed conns, decrease buffer len

few doc cleanups

Further memory conservation by making state bitfield :8.

Does this need #ifdef __GCC__?

log proper reason for disconnect

remove extra argument for SBuf callback

conserves memory and conforms more with general coding style.

plus few cleanups

dont waste room for storing login-time salt.

Instead store it in cancel_key[], as it is used
only once, before generating the cancel_key.

decrease noise from bad connections

the accepting new connection does not need to stop if bad one appears.

keep mtime in tgz

duplicate tagline

require libevent >= 1.3b