aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/core/lib
diff options
context:
space:
mode:
authorGravatar Yash Tibrewal <yashkt@google.com>2018-11-29 10:00:57 -0800
committerGravatar Yash Tibrewal <yashkt@google.com>2018-11-29 10:00:57 -0800
commitba45e77413b0475036bf5d22adabf8bd432fd443 (patch)
tree23049cebf915b634cd7354ff2bc437c07d87e193 /src/core/lib
parentaf16b2c09d75f56acc9fa2c7d76ebb038e06ea3e (diff)
Revert the do while and if
Diffstat (limited to 'src/core/lib')
-rw-r--r--src/core/lib/iomgr/tcp_posix.cc17
1 files changed, 9 insertions, 8 deletions
diff --git a/src/core/lib/iomgr/tcp_posix.cc b/src/core/lib/iomgr/tcp_posix.cc
index 84d593426e..68c61b1201 100644
--- a/src/core/lib/iomgr/tcp_posix.cc
+++ b/src/core/lib/iomgr/tcp_posix.cc
@@ -732,12 +732,9 @@ static void process_errors(grpc_tcp* tcp) {
/* There was no control message found. It was probably spurious. */
return;
}
- auto cmsg = CMSG_FIRSTHDR(&msg);
- if (cmsg == nullptr || cmsg->cmsg_len == 0) {
- /* No control message found. */
- return;
- }
- do {
+ bool seen = false;
+ for (auto cmsg = CMSG_FIRSTHDR(&msg); cmsg && cmsg->cmsg_len;
+ cmsg = CMSG_NXTHDR(&msg, cmsg)) {
if (cmsg->cmsg_level != SOL_SOCKET ||
cmsg->cmsg_type != SCM_TIMESTAMPING) {
/* Got a control message that is not a timestamp. Don't know how to
@@ -749,8 +746,12 @@ static void process_errors(grpc_tcp* tcp) {
}
return;
}
- cmsg = CMSG_NXTHDR(&msg, process_timestamp(tcp, &msg, cmsg));
- } while (cmsg && cmsg->cmsg_len);
+ cmsg = process_timestamp(tcp, &msg, cmsg);
+ seen = true;
+ }
+ if (!seen) {
+ return;
+ }
}
}