Re: [xml] [PATCH] Two trivial fixes for Control Flow Integrity support

[Nick Wellnhofer <wellnhofer aevum de>]

On 09/11/2017 01:18, Vlad Tsyrklevich wrote:
> I've updated the patches.

The patches are now applied, the second one with a minor change.

I also fixed quite a few other callback signatures to make the test suite pass 
with CFI checks:

https://git.gnome.org/browse/libxml2/commit/?id=e03f0a199a67017b2f8052354cf732b2b4cae787
https://git.gnome.org/browse/libxml2/commit/?id=4dd6d7a58e3694af7ed42589affcfa9a1550dada
https://git.gnome.org/browse/libxml2/commit/?id=e5f33e56bafc332fd63f6643b687ebfe14bcba56
https://git.gnome.org/browse/libxml2/commit/?id=86615e43bbac2315aa069ca3ef4712477d61605c

I used the following configure command on Ubuntu 17.04:

    CC=clang \
    CFLAGS="-fsanitize=cfi -flto -fvisibility=hidden -fno-sanitize-trap=all -g" \
    RANLIB=llvm-ranlib \
    ./configure --disable-shared

Nick