https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67567

--- Comment #4 from Manuel López-Ibáñez <manu at gcc dot gnu.org> ---
Author: manu
Date: Fri Sep 25 14:24:11 2015
New Revision: 228131

URL: https://gcc.gnu.org/viewcvs?rev=228131&root=gcc&view=rev
Log:
PR pretty-print/67567 do not pass NULL as a string

Fortran passes NULL where a non-null string is expected by the pretty-printer,
which causes a sanitizer warning. This could have been found earlier by using
gcc_checking_assert. Even if the assertion is false, the result is just an
incomplete diagnostic, thus it seems more user-friendly to assert only when
checking. I do not have any idea how to properly fix the Fortran bug, thus this
patch simply works-around it.

gcc/fortran/ChangeLog:

2015-09-25  Manuel López-Ibáñez  <m...@gcc.gnu.org>

        PR pretty-print/67567
        * resolve.c (resolve_fl_procedure): Work-around when iface->module
        == NULL.

gcc/ChangeLog:

2015-09-25  Manuel López-Ibáñez  <m...@gcc.gnu.org>

        PR pretty-print/67567
        * pretty-print.c (pp_string): Add gcc_checking_assert.
        * pretty-print.h (output_buffer_append_r): Likewise.


Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/fortran/ChangeLog
    trunk/gcc/fortran/resolve.c
    trunk/gcc/pretty-print.c
    trunk/gcc/pretty-print.h

Reply via email to