commit | b406b8708fef5ff51e684794b5df4fea728b0058 | [log] [tgz] |
---|---|---|
author | Willy Tarreau <w@1wt.eu> | Wed Aug 22 05:20:32 2018 +0200 |
committer | Willy Tarreau <w@1wt.eu> | Wed Aug 22 05:28:32 2018 +0200 |
tree | 686b5609f484c1d15d2517159f83a5b0b31f8620 | |
parent | 2842e05c7c9d9fb7e03f8900f778dc4fae4deb78 [diff] |
BUG/MEDIUM: connection: don't store recv() result into trash.data Cyril Bonté discovered that the proxy protocol randomly fails since commit 843b7cb ("MEDIUM: chunks: make the chunk struct's fields match the buffer struct"). This is because we used to store recv()'s return code into trash.data which is now unsigned, so it never compares as negative against 0. Let's clean this up and test the result itself without storing it first. No backport is needed.