plaisthos has uploaded this change for review. (
http://gerrit.openvpn.net/c/openvpn/+/1483?usp=email )
Change subject: Change stream_buf_read_setup_dowork parameter to struct
steam_buf
......................................................................
Change stream_buf_read_setup_dowork parameter to struct steam_buf
This methods only ever access sock->stream_buf so make the method
simpler by just having a parameter sb.
Change-Id: I3deb7cd75db3cb280fa8d9c637cd3bde3881d6e3
---
M src/openvpn/socket.c
M src/openvpn/socket.h
2 files changed, 16 insertions(+), 11 deletions(-)
git pull ssh://gerrit.openvpn.net:29418/openvpn refs/changes/83/1483/1
diff --git a/src/openvpn/socket.c b/src/openvpn/socket.c
index 331249d..f8c0b5c 100644
--- a/src/openvpn/socket.c
+++ b/src/openvpn/socket.c
@@ -2123,22 +2123,22 @@
}
bool
-stream_buf_read_setup_dowork(struct link_socket *sock)
+stream_buf_read_setup_dowork(struct stream_buf *sb)
{
- if (sock->stream_buf.residual.len &&
!sock->stream_buf.residual_fully_formed)
+ if (sb->residual.len && !sb->residual_fully_formed)
{
- ASSERT(buf_copy(&sock->stream_buf.buf, &sock->stream_buf.residual));
- ASSERT(buf_init(&sock->stream_buf.residual, 0));
- sock->stream_buf.residual_fully_formed =
stream_buf_added(&sock->stream_buf, 0);
+ ASSERT(buf_copy(sb->buf, &sock->stream_buf.residual));
+ ASSERT(buf_init(&sb->residual, 0));
+ sb->residual_fully_formed = stream_buf_added(sb, 0);
dmsg(D_STREAM_DEBUG, "STREAM: RESIDUAL FULLY FORMED [%s], len=%d",
- sock->stream_buf.residual_fully_formed ? "YES" : "NO",
sock->stream_buf.residual.len);
+ stream_buf->residual_fully_formed ? "YES" : "NO",
sb->residual.len);
}
- if (!sock->stream_buf.residual_fully_formed)
+ if (!sb->residual_fully_formed)
{
- stream_buf_set_next(&sock->stream_buf);
+ stream_buf_set_next(sb);
}
- return !sock->stream_buf.residual_fully_formed;
+ return !sb->residual_fully_formed;
}
static bool
diff --git a/src/openvpn/socket.h b/src/openvpn/socket.h
index 850064d..8664666f 100644
--- a/src/openvpn/socket.h
+++ b/src/openvpn/socket.h
@@ -528,14 +528,19 @@
}
}
-bool stream_buf_read_setup_dowork(struct link_socket *sock);
+/**
+ *
+ * @param sock
+ * @return
+ */
+bool stream_buf_read_setup_dowork(struct stream_buf *sb);
static inline bool
stream_buf_read_setup(struct link_socket *sock)
{
if (link_socket_connection_oriented(sock))
{
- return stream_buf_read_setup_dowork(sock);
+ return stream_buf_read_setup_dowork(&sock->stream_buf);
}
else
{
--
To view, visit http://gerrit.openvpn.net/c/openvpn/+/1483?usp=email
To unsubscribe, or for help writing mail filters, visit
http://gerrit.openvpn.net/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: openvpn
Gerrit-Branch: master
Gerrit-Change-Id: I3deb7cd75db3cb280fa8d9c637cd3bde3881d6e3
Gerrit-Change-Number: 1483
Gerrit-PatchSet: 1
Gerrit-Owner: plaisthos <[email protected]>
Gerrit-CC: openvpn-devel <[email protected]>
_______________________________________________
Openvpn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
