GitHub user jdanekrh opened a pull request:

    https://github.com/apache/qpid-dispatch/pull/189

    DISPATCH-809 Add options to enable Sanitizers to CMake build

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/jdanekrh/qpid-dispatch jd_sanitizers

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/qpid-dispatch/pull/189.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #189
    
----
commit 34c70f703c09b1883ea125a3d0bbc4b528f6e535
Author: Jiri Danek <[email protected]>
Date:   2017-08-14T16:54:36Z

    NO-JIRA Fix alloc-size-larger-than warning from gcc 7
    
    The following warning is fixed by this commit
    
    ```
    /home/jdanek/Work/repos/qpid-dispatch/src/entity.c: In function 
‘qd_entity_set_stringf’:
    /home/jdanek/Work/repos/qpid-dispatch/src/entity.c:235:10: error: argument 
1 range [18446744071562067968, 18446744073709551615] exceeds maximum object 
size 9223372036854775807 [-Werror=alloc-size-larger-than=]
         char buf[len+1];
              ^~~
    /home/jdanek/Work/repos/qpid-dispatch/src/entity.c:235:10: note: in a call 
to built-in allocation function ‘__builtin_alloca_with_align’
    /home/jdanek/Work/repos/qpid-dispatch/src/entity.c:235:10: error: argument 
1 range [18446744071562067968, 18446744073709551615] exceeds maximum object 
size 9223372036854775807 [-Werror=alloc-size-larger-than=]
    /home/jdanek/Work/repos/qpid-dispatch/src/entity.c:235:10: note: in a call 
to built-in allocation function ‘__builtin_alloca_with_align’
    cc1: all warnings being treated as errors
    make[2]: *** [src/CMakeFiles/qpid-dispatch.dir/build.make:273: 
src/CMakeFiles/qpid-dispatch.dir/entity.c.o] Error 1
    make[2]: *** Waiting for unfinished jobs....
    make[1]: *** [CMakeFiles/Makefile2:982: 
src/CMakeFiles/qpid-dispatch.dir/all] Error 2
    make: *** [Makefile:141: all] Error 2
    ```

commit 1e9708be2fc79540c3228affd677ae85648be8b2
Author: Jiri Danek <[email protected]>
Date:   2017-08-14T19:48:18Z

    NO-JIRA Silence clang 4 error: use of GNU statement expression extension
    
    This fixes the following kind of warnings
    
    /home/jdanek/Work/repos/qpid-dispatch/src/posix/threading.c:147:5: error: 
use of GNU statement expression extension [-Werror,-Wgnu-statement-expression]
        assert(result == 0);
        ^
    
/nix/store/qy94v105wag3z9rgy1rb34zk0x20lkwj-glibc-2.25-dev/include/assert.h:95:6:
 note: expanded from macro 'assert'
        ({                                                                  \
         ^

commit de16048f3b577dbf7fc831475d02acaa9581e543
Author: Jiri Danek <[email protected]>
Date:   2017-08-14T19:36:50Z

    DISPATCH-809 Add options to enable Sanitizers to CMake build
    
    Tested to work with gcc 5.4.0, 7.1.0 and clang 4.0.1

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to