1. 07 Jun, 2022 6 commits
  2. 02 Jun, 2022 3 commits
    • Carlos O'Donell's avatar
      We should not free the nrhs if it was not set. · 6788224a
      Carlos O'Donell authored
      
      
      Fix compilation with gcc 11 and glibc 2.33.
      
      It is possible to have a switch statement value that causes an
      invalid free. So set nrhs to a value that can be distinguished if
      we are in the fall-through case.
      
      expr.c: In function ‘expr_clone’:
      expr.c:240:33: error: ‘nrhs’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
        240 |                                 free(nrhs);
            |                                 ^~~~~~~~~~
      cc1: all warnings being treated as errors
      
      This is downstream Fedora patch: ltrace-0.7.91-rh1799619.patch
      The downstream patch can be removed.
      Signed-off-by: Carlos O'Donell's avatarCarlos O'Donell <carlos@redhat.com>
      6788224a
    • Carlos O'Donell's avatar
      Use a default value for the printed module and symbol name. · 272675b3
      Carlos O'Donell authored
      
      
      Fix compiling with gcc 11 and glibc 2.33 by using a default value for
      the module and symbol name if we can't look them up. We use "[unknown]"
      as the default.
      
      output.c: In function ‘frame_callback’:
      output.c:660:9: error: ‘%s’ directive argument is null [-Werror=format-overflow=]
        660 |         fprintf(options.output, " > %s(%s+0x%" PRIx64 ") [%" PRIx64 "]\n",
            |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
        661 |                 modname, symname, off, pc);
            |                 ~~~~~~~~~~~~~~~~~~~~~~~~~~
      cc1: all warnings being treated as errors
      
      This is downstream Fedora patch: ltrace-0.7.91-null.patch
      This patch can be now be removed.
      Signed-off-by: Carlos O'Donell's avatarCarlos O'Donell <carlos@redhat.com>
      272675b3
    • Carlos O'Donell's avatar
      linux: Fix compiling proc with gcc 11 and glibc 2.33. · 5d9045e5
      Carlos O'Donell authored
      
      
      Switch to using readdir to fix the use of a deprecated function.
      
      proc.c: In function ‘process_tasks’:
      proc.c:247:17: error: ‘readdir_r’ is deprecated [-Werror=deprecated-declarations]
        247 |                 if (readdir_r(d, &entry, &result) != 0) {
            |                 ^~
      In file included from proc.c:31:
      /usr/include/dirent.h:183:12: note: declared here
        183 | extern int readdir_r (DIR *__restrict __dirp,
            |            ^~~~~~~~~
      cc1: all warnings being treated as errors
      
      This is downstream Fedora patch: ltrace-0.7.91-tautology.patch
      without the removal of the forced asserts. The downstream patches
      to fix this can be removed.
      Signed-off-by: Carlos O'Donell's avatarCarlos O'Donell <carlos@redhat.com>
      5d9045e5
  3. 11 Nov, 2015 31 commits