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:46:55 +0000 (08:46 +0200)
commite44d171b70bd7c4494fe8cc5d068b6cdd47a2c22
tree7adbc1cdd38c3ae851e0e7f10fa9f378f344b48a
parent84ed18b5d99abd178a54c181ba853097712d7acb
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