Properly reject invalid handshake data
authorManuel Nickschas <sputnick@quassel-irc.org>
Tue, 21 Sep 2010 06:46:55 +0000 (08:46 +0200)
committerManuel Nickschas <sputnick@quassel-irc.org>
Tue, 21 Sep 2010 06:57:48 +0000 (08:57 +0200)
commit09a329904280209cd6a4b9d9dbd6590167fb2146
tree3619f479dc479c5ddb36a88aa68e425a234bc12c
parent1999204bf07260c3856194c13f60ef463fcd1141
Properly reject invalid handshake data

Currently, if a client attempts to configure a core's storage backend
even though it is already configured, the core will ignore the attempt
silently (e.g. the existing storage backends will simply drop the request,
but the client also won't receive an error message). This can never happen
unless you use an alternative client with a buggy handshake sequence though,
since quasselclient checks if the core is already configured beforehand.

Nonetheless, it makes sense to properly error out from the core side, just in case
we're getting new storage backends in the future that don't properly check this,
or alternative clients which might be confused by not receiving an error when they should.

Thanks to JodaZ for pointing this out.
src/core/core.cpp