cron2 has submitted this change. ( http://gerrit.openvpn.net/c/openvpn/+/1107?usp=email )
Change subject: events: Make sure rwflags are treated as unsigned ...................................................................... events: Make sure rwflags are treated as unsigned event_set_return.rwflags is already unsigned, make sure the flags are as well to avoid spurious conversion warnings. Requires to change rwflags in proxy_connection struct as well since those use the same flags. Change-Id: I0272b709b907545de05bfded03a649b259ce5af6 Signed-off-by: Frank Lichtenheld <[email protected]> Acked-by: Gert Doering <[email protected]> URL: https://gerrit.openvpn.net/c/openvpn/+/1107 Message-Id: <[email protected]> URL: https://www.mail-archive.com/[email protected]/msg32918.html Signed-off-by: Gert Doering <[email protected]> --- M src/openvpn/event.h M src/openvpn/ps.c 2 files changed, 10 insertions(+), 9 deletions(-) diff --git a/src/openvpn/event.h b/src/openvpn/event.h index 8422996..8a89a25 100644 --- a/src/openvpn/event.h +++ b/src/openvpn/event.h @@ -35,8 +35,8 @@ #define WRITE_SHIFT 1 #define EVENT_UNDEF 4 -#define EVENT_READ (1 << READ_SHIFT) -#define EVENT_WRITE (1 << WRITE_SHIFT) +#define EVENT_READ (1u << READ_SHIFT) +#define EVENT_WRITE (1u << WRITE_SHIFT) /* event flags returned by io_wait. * diff --git a/src/openvpn/ps.c b/src/openvpn/ps.c index e95a33b..a10df2e 100644 --- a/src/openvpn/ps.c +++ b/src/openvpn/ps.c @@ -70,7 +70,7 @@ struct proxy_connection *counterpart; struct buffer buf; bool buffer_initial; - int rwflags; + unsigned int rwflags; int sd; char *jfn; }; @@ -387,12 +387,12 @@ } static inline void -proxy_connection_io_requeue(struct proxy_connection *pc, const int rwflags_new, +proxy_connection_io_requeue(struct proxy_connection *pc, const unsigned int rwflags_new, struct event_set *es) { if (socket_defined(pc->sd) && pc->rwflags != rwflags_new) { - /*dmsg (D_PS_PROXY_DEBUG, "PORT SHARE PROXY: requeue[%d] rwflags=%d", (int)pc->sd, + /*dmsg (D_PS_PROXY_DEBUG, "PORT SHARE PROXY: requeue[%d] rwflags=%u", (int)pc->sd, * rwflags_new);*/ event_ctl(es, pc->sd, rwflags_new, (void *)pc); pc->rwflags = rwflags_new; @@ -652,7 +652,7 @@ * Decide how the receipt of an EAGAIN status should affect our next IO queueing. */ static bool -proxy_connection_io_status(const int status, int *rwflags_pc, int *rwflags_cp) +proxy_connection_io_status(const int status, unsigned int *rwflags_pc, unsigned int *rwflags_cp) { switch (status) { @@ -683,12 +683,13 @@ * in the proxied connection. */ static int -proxy_connection_io_dispatch(struct proxy_connection *pc, const int rwflags, struct event_set *es) +proxy_connection_io_dispatch(struct proxy_connection *pc, const unsigned int rwflags, + struct event_set *es) { const int max_transfer_per_iteration = 10000; struct proxy_connection *cp = pc->counterpart; - int rwflags_pc = pc->rwflags; - int rwflags_cp = cp->rwflags; + unsigned int rwflags_pc = pc->rwflags; + unsigned int rwflags_cp = cp->rwflags; ASSERT(pc->defined && cp->defined && cp->counterpart == pc); -- To view, visit http://gerrit.openvpn.net/c/openvpn/+/1107?usp=email To unsubscribe, or for help writing mail filters, visit http://gerrit.openvpn.net/settings Gerrit-Project: openvpn Gerrit-Branch: master Gerrit-Change-Id: I0272b709b907545de05bfded03a649b259ce5af6 Gerrit-Change-Number: 1107 Gerrit-PatchSet: 11 Gerrit-Owner: flichtenheld <[email protected]> Gerrit-Reviewer: cron2 <[email protected]> Gerrit-Reviewer: plaisthos <[email protected]> Gerrit-CC: openvpn-devel <[email protected]> Gerrit-MessageType: merged
_______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
