testbot/TestAgent: Remove some unneeded checks for a broken connection.

[Francois Gouget]

These checks are redundant with the first recv_xxx() or send_xxx() call these functions make (only send_file() is a slightly special case).
---
 testbot/src/testagentd/testagentd.c |   12 ------------
 1 file changed, 12 deletions(-)

diff --git a/testbot/src/testagentd/testagentd.c b/testbot/src/testagentd/testagentd.c
index 607c776..a548491 100644
--- a/testbot/src/testagentd/testagentd.c
+++ b/testbot/src/testagentd/testagentd.c
@@ -217,9 +217,6 @@ static int recv_raw_data(SOCKET client, void* data, uint64_t size)
 
 static int recv_raw_uint32(SOCKET client, uint32_t *u32)
 {
-    if (broken)
-        return 0;
-
     if (!recv_raw_data(client, u32, sizeof(*u32)))
         return 0;
     *u32 = ntohl(*u32);
@@ -230,9 +227,6 @@ static int recv_raw_uint64(SOCKET client, uint64_t *u64)
 {
     uint32_t high, low;
 
-    if (broken)
-        return 0;
-
     if (!recv_raw_uint32(client, &high) || !recv_raw_uint32(client, &low))
         return 0;
     *u64 = ((uint64_t)high) << 32 | low;
@@ -246,9 +240,6 @@ static int recv_raw_uint64(SOCKET client, uint64_t *u64)
 
 static int recv_entry_header(SOCKET client, char *type, uint64_t *size)
 {
-    if (broken)
-        return 0;
-
     return recv_raw_data(client, type, sizeof(*type)) &&
            recv_raw_uint64(client, size);
 }
@@ -261,9 +252,6 @@ static int expect_entry_header(SOCKET client, char type, uint64_t *size)
     uint64_t hsize;
     int success;
 
-    if (broken)
-        return 0;
-
     if (!recv_entry_header(client, &htype, &hsize))
         return 0;
 
-- 
1.7.10.4