The branch, master has been updated
via ce38138 swrap: Fix comparison of different signedness
from ac18c80 Bump version to 1.2.0
https://git.samba.org/?p=socket_wrapper.git;a=shortlog;h=master
- Log -----------------------------------------------------------------
commit ce38138a9ac1b9e36b3338ee9ebcdc5686d541a4
Author: Andreas Schneider <a...@samba.org>
Date: Tue Nov 13 14:28:17 2018 +0100
swrap: Fix comparison of different signedness
Found by csbuild.
Signed-off-by: Andreas Schneider <a...@samba.org>
Reviewed-by: Ralph Boehme <s...@samba.org>
-----------------------------------------------------------------------
Summary of changes:
src/socket_wrapper.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
Changeset truncated at 500 lines:
diff --git a/src/socket_wrapper.c b/src/socket_wrapper.c
index 2b77ceb..ae3b2b0 100644
--- a/src/socket_wrapper.c
+++ b/src/socket_wrapper.c
@@ -1539,7 +1539,7 @@ static int find_socket_info_index(int fd)
return -1;
}
- if (fd >= socket_fds_max) {
+ if ((size_t)fd >= socket_fds_max) {
SWRAP_LOG(SWRAP_LOG_ERROR,
"The max socket index limit of %zu has been reached, "
"trying to add %d",
@@ -1590,7 +1590,7 @@ static int swrap_create_socket(struct socket_info *si,
int fd)
{
int idx;
- if (fd >= socket_fds_max) {
+ if ((size_t)fd >= socket_fds_max) {
SWRAP_LOG(SWRAP_LOG_ERROR,
"The max socket index limit of %zu has been reached, "
"trying to add %d",
--
Socket Wrapper Repository
