[OpenSIPS-Devel] [OpenSIPS/opensips] 53fa21: siprec: fix dialog callback ref ownership

[Răzvan Crainea]

  Branch: refs/heads/master
  Home:   https://github.com/OpenSIPS/opensips
  Commit: 53fa21afe140283aafc1df5277f7dcd1e898b2ee
      https://github.com/OpenSIPS/opensips/commit/53fa21afe140283aafc1df5277f7dcd1e898b2ee
  Author: Razvan Crainea <razvan at opensips.org>
  Date:   2026-08-18 (Tue, 18 Aug 2026)

  Changed paths:
    M modules/siprec/siprec_logic.c

  Log Message:
  -----------
  siprec: fix dialog callback ref ownership

Set SIPREC_DLG_CBS when the dialog termination callback takes ownership of the session ref.

Thanks to @kimaiden1984-boop for the report.

Closes #4132


  Commit: 36b28c72bb17d1b11babfabac664d85338b26ef8
      https://github.com/OpenSIPS/opensips/commit/36b28c72bb17d1b11babfabac664d85338b26ef8
  Author: Yury Kirsanov <yury at kfi.net.au>
  Date:   2026-08-18 (Tue, 18 Aug 2026)

  Changed paths:
    M lib/url.c

  Log Message:
  -----------
  lib/url.c: keep _GNU_SOURCE defined for musl libc builds

url.c defined _GNU_SOURCE only around <stdio.h>/<string.h> and then
undefined it again before including ut.h. On musl libc the feature-test
macros are re-evaluated by every system header, so once _GNU_SOURCE is
removed the later <time.h> (pulled in through ut.h and dprint.h) no
longer exposes clock_gettime(), CLOCK_REALTIME or ctime_r(). The core
then fails to compile on Alpine/musl with gcc 15:

    lib/../mem/../dprint.h:204: implicit declaration of 'ctime_r'
    lib/../ut.h:1401: implicit declaration of 'clock_gettime'
    lib/../ut.h:1401: 'CLOCK_REALTIME' undeclared

glibc is unaffected: it latches internal __USE_* macros at the first
header inclusion, which the later #undef does not clear.

Keep _GNU_SOURCE defined for the whole translation unit, matching how
other core files (io_wait.h, transformations.c) enable it.


  Commit: 5abccf465faebaa76405f983d6b54c2d75d8892b
      https://github.com/OpenSIPS/opensips/commit/5abccf465faebaa76405f983d6b54c2d75d8892b
  Author: Yury Kirsanov <yury at kfi.net.au>
  Date:   2026-08-18 (Tue, 18 Aug 2026)

  Changed paths:
    M modules/mathops/math_funcs.c
    M modules/mathops/mathops.c

  Log Message:
  -----------
  modules/mathops: keep the feature-test macros defined for musl libc

Both mathops sources define _XOPEN_SOURCE/_GNU_SOURCE for <math.h>, then
undefine them again before including the OpenSIPS headers - the same
pattern lib/url.c had.

musl re-evaluates the feature-test macros in every system header rather
than only on the first one, and its features.h turns on the permissive
default set only when no source macro is defined at all.  Once the block
above has defined them, undefining them leaves the translation unit with
none of them set, so <time.h> stops declaring ctime_r() and dprint.h no
longer compiles:

  Compiling mathops.c
  ../../parser/../dprint.h:204:9: error: implicit declaration of function
      'ctime_r'; did you mean 'ctime'?

Since gcc 14 an implicit declaration is an error, so this is a hard build
failure of the module on Alpine.  Keep the macros defined for the rest of
each file, as lib/url.c now does.


  Commit: 4f65354b05d178826cec716d6da4d8f9e6bda209
      https://github.com/OpenSIPS/opensips/commit/4f65354b05d178826cec716d6da4d8f9e6bda209
  Author: Razvan Crainea <razvan at opensips.org>
  Date:   2026-08-18 (Tue, 18 Aug 2026)

  Changed paths:
    M msg_translator.c

  Log Message:
  -----------
  msg_translator: handle empty multipart bodies

Use a derived zero-length body range when a received body part has no body pointer, keeping multipart reassembly from doing pointer arithmetic on NULL.

Security advisory: ANT-2026-CKSJ0NQF

Thanks to Claude and Ada Logics for the report.


Compare: https://github.com/OpenSIPS/opensips/compare/e12c990d2587...4f65354b05d1

To unsubscribe from these emails, change your notification settings at https://github.com/OpenSIPS/opensips/settings/notifications