Minor code change ('return' -> 'break').

src/core/or/safe_connection.c

@@ -1736,7 +1736,7 @@ process_cells_from_inbuf(safe_or_connection_t *safe_or_conn)
     if (found_var_cell) {
       if (var_cell == NULL) {
         // the next cell is a var cell, but it is not yet complete
-        return;
+        break;
       }
 
       uint8_t command = var_cell->command;
@@ -1753,7 +1753,7 @@ process_cells_from_inbuf(safe_or_connection_t *safe_or_conn)
         // in order to process future cells, we need to be told our
         // protocol version
         safe_or_conn->waiting_for_link_protocol = true;
-        return;
+        break;
       }
     } else {
       char buf[CELL_MAX_NETWORK_SIZE];
@@ -1773,7 +1773,7 @@ process_cells_from_inbuf(safe_or_connection_t *safe_or_conn)
         // we no longer own the cell at this point, so don't access it again
       } else {
         // there is not yet a complete cell
-        return;
+        break;
       }
     }
   }