[SCM] Socket Wrapper Repository - branch master updated

Andreas Schneider asn at samba.org
Fri Oct 24 06:52:02 MDT 2014


The branch, master has been updated
       via  f05cbad swrap: Do not leak the socket_info we just removed.
       via  f0606cf cmake: Treat no_sanitize_address attribute warnings as error.
      from  db8edc8 src: Add support for running with address sanitizer.

http://gitweb.samba.org/?p=socket_wrapper.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit f05cbad1929ad0f144834b0df444239ab0c05a21
Author: Andreas Schneider <asn at samba.org>
Date:   Fri Oct 24 11:22:15 2014 +0200

    swrap: Do not leak the socket_info we just removed.
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Michael Adam <obnox at samba.org>

commit f0606cf44ac93c2105296d608f5e88ec0a9ba3c9
Author: Andreas Schneider <asn at samba.org>
Date:   Thu Oct 23 07:23:36 2014 +0200

    cmake: Treat no_sanitize_address attribute warnings as error.
    
    Signed-off-by: Andreas Schneider <asn at samba.org>
    Reviewed-by: Michael Adam <obnox at samba.org>

-----------------------------------------------------------------------

Summary of changes:
 ConfigureChecks.cmake |    3 +++
 src/socket_wrapper.c  |    1 +
 2 files changed, 4 insertions(+), 0 deletions(-)


Changeset truncated at 500 lines:

diff --git a/ConfigureChecks.cmake b/ConfigureChecks.cmake
index 32cd9bb..71e5056 100644
--- a/ConfigureChecks.cmake
+++ b/ConfigureChecks.cmake
@@ -194,6 +194,8 @@ int main(void) {
     return 0;
 }" HAVE_FUNCTION_ATTRIBUTE_FORMAT)
 
+# If this produces a warning treat it as error!
+set(CMAKE_REQUIRED_FLAGS "-Werror")
 check_c_source_compiles("
 void test_address_sanitizer_attribute(void) __attribute__((no_sanitize_address));
 
@@ -205,6 +207,7 @@ void test_address_sanitizer_attribute(void)
 int main(void) {
     return 0;
 }" HAVE_ADDRESS_SANITIZER_ATTRIBUTE)
+set(CMAKE_REQUIRED_FLAGS)
 
 check_library_exists(dl dlopen "" HAVE_LIBDL)
 if (HAVE_LIBDL)
diff --git a/src/socket_wrapper.c b/src/socket_wrapper.c
index 82e5737..eb1d67f 100644
--- a/src/socket_wrapper.c
+++ b/src/socket_wrapper.c
@@ -1418,6 +1418,7 @@ static void swrap_remove_stale(int fd)
 
 		if (si->fds == NULL) {
 			SWRAP_DLIST_REMOVE(sockets, si);
+			free(si);
 		}
 	}
 }


-- 
Socket Wrapper Repository


More information about the samba-cvs mailing list