• Show log

    Commit

  • Hash : 98a6d9d5
    Author : Noah Pendleton
    Date : 2018-11-13T08:22:25

    tests: address two null argument instances
    
    Handle two null argument cases that occur in the unit tests.
    One is in library code, the other is in test code.
    
    Detected by running unit tests with undefined behavior sanitizer:
    ```bash
     # build
    mkdir build && cd build
    cmake -DBUILD_CLAR=ON -DCMAKE_C_FLAGS="-fsanitize=address \
    -fsanitize=undefined -fstack-usage -static-libasan" ..
    cmake --build .
    
     # run with asan
    ASAN_OPTIONS="allocator_may_return_null=1" ./libgit2_clar
    ...
    ............../libgit2/src/apply.c:316:3: runtime error: null pointer \
    passed as argument 1, which is declared to never be null
    ...................../libgit2/tests/apply/fromfile.c:46:3: runtime \
    error: null pointer passed as argument 1, which is declared to never be null
    ```