ctdb: Remove double sanity checks from ctdb_tcp_read_cb

Within ctdb_tcp_read_cb the provided data is checked for sanity,
e.g. correct size and content. This is not required because it was
done already by the caller(queue_process).

Signed-off-by: Swen Schillig <swen@vnet.ibm.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Martin Schwenke <martins@samba.org>
Autobuild-Date(master): Wed Apr  4 09:31:04 CEST 2018 on sn-devel-144

diff --git a/ctdb/tcp/tcp_io.c b/ctdb/tcp/tcp_io.c
index 3c3b1c167f320f7c4517aa7b0a70de4cb1ada153..0eb8e25eea3aa80dfefa27a67338eb5ae908cbe1 100644 (file)
--- a/ctdb/tcp/tcp_io.c
+++ b/ctdb/tcp/tcp_io.c
@@ -56,13 +56,6 @@ void ctdb_tcp_read_cb(uint8_t *data, size_t cnt, void *args)
                goto failed;
        }
 
-
-       if (cnt != hdr->length) {
-               DEBUG(DEBUG_ALERT,(__location__ " Bad header length %u expected %u\n", 
-                        (unsigned)hdr->length, (unsigned)cnt));
-               goto failed;
-       }
-
        if (hdr->ctdb_magic != CTDB_MAGIC) {
                DEBUG(DEBUG_ALERT,(__location__ " Non CTDB packet 0x%x rejected\n", 
                         hdr->ctdb_magic));