[OpenSIPS-Devel] [OpenSIPS/opensips] 0147ba: Convert _str() from inline to a macro and make it ...

Maksym Sobolyev noreply at github.com
Thu Jan 28 19:47:25 EST 2021


  Branch: refs/heads/master
  Home:   https://github.com/OpenSIPS/opensips
  Commit: 0147baa21072c5b9486a0261df49a89a1c14f16c
      https://github.com/OpenSIPS/opensips/commit/0147baa21072c5b9486a0261df49a89a1c14f16c
  Author: Maksym Sobolyev <sobomax at sippysoft.com>
  Date:   2021-01-28 (Thu, 28 Jan 2021)

  Changed paths:
    M modules/dialog/dlg_handlers.c
    M modules/dialog/dlg_handlers.h
    M modules/dialog/dlg_hash.c
    M modules/dialog/dlg_hash.h
    M modules/dialog/dlg_load.h
    M modules/dialplan/dialplan.c
    M str.h

  Log Message:
  -----------
  Convert _str() from inline to a macro and make it return
(const str *), not (str *). This is to limit potential
damage from its (mis)use in non-testing modules code by
first making its value stored locally at point of use
(which should allow compiler to throw it away when used
properly) and also provide at least some protection from
any function from messing with its fields.

Fix any fallout where str * has been passed around.


  Commit: 2b455a392b7eea9c67c845c990a1cb336af14c34
      https://github.com/OpenSIPS/opensips/commit/2b455a392b7eea9c67c845c990a1cb336af14c34
  Author: Maksym Sobolyev <sobomax at sippysoft.com>
  Date:   2021-01-28 (Thu, 28 Jan 2021)

  Changed paths:
    M modules/rtpproxy/rtpproxy_vcmd.h

  Log Message:
  -----------
  Set .iov_base = NULL for cookie and terminator when initializing
on stack. This is to silence compiler warnings on Centos 7.


Compare: https://github.com/OpenSIPS/opensips/compare/d85d0b50ec63...2b455a392b7e



More information about the Devel mailing list