src/*.c: Always use constants on the left in comparisons.

diff --git a/src/connection.c b/src/connection.c
index 4160c7feff77388206b3459bf82f18c8f2734eeb..a42fbe055ae9349094743344c9a5c814efc2b584 100644 (file)
--- a/src/connection.c
+++ b/src/connection.c
@@ -71,10 +71,10 @@ void handle_connection(int client_socket) {
 
     /* Read request line (CONNECT ..) and headers (they are discarded). */
     result = read_http_request(client_fd, buffer, sizeof(buffer));
-    if (result == -1) {
+    if (-1 == result) {
         /* Read error. */
         return;
-    } else if (result == -2) {
+    } else if (-2 == result) {
         /* EOF */
         send_close_bad_request(client_fd);
         return;
@@ -120,11 +120,11 @@ void handle_connection(int client_socket) {
 
         /* Read response line from proxy server. */
         result = read_http_request(server_fd, buffer, sizeof(buffer));
-        if (result == -1) {
+        if (-1 == result) {
             /* Read error. */
             send_close_forwarding_failure(client_fd);
             return;
-        } else if (result == -2) {
+        } else if (-2 == result) {
             /* EOF */
             fclose(server_fd);
             send_close_forwarding_failure(client_fd);

diff --git a/src/tlsproxy.c b/src/tlsproxy.c
index 9c3b53c26383782eadde7028ad295cb8c40afc1b..7c7013383e3bf948c75742c6327a7308d0c8e4c3 100644 (file)
--- a/src/tlsproxy.c
+++ b/src/tlsproxy.c
@@ -291,7 +291,7 @@ static void worker_thread(void) {
         V(ringbuffer_free);
 
         /* Negative value indicates we should shut down our thread. */
-        if (client_socket < 0) {
+        if (0 > client_socket) {
             break;
         }