[OpenSIPS-Devel] [OpenSIPS/opensips] 8b1806: permissions: Fix incorrect match of IPv4 addr agai...

[Liviu Chircu]

  Branch: refs/heads/master
  Home:   https://github.com/OpenSIPS/opensips
  Commit: 8b1806ec667821eab25c0d79147a8b9cf5320074
      https://github.com/OpenSIPS/opensips/commit/8b1806ec667821eab25c0d79147a8b9cf5320074
  Author: Liviu Chircu <liviu at opensips.org>
  Date:   2025-09-16 (Tue, 16 Sep 2025)

  Changed paths:
    M modules/permissions/hash.c

  Log Message:
  -----------
  permissions: Fix incorrect match of IPv4 addr against IPv6

matchnet()'s possible rcs were not fully handled (here, the -1 case).

Many thanks to Bernard Buitenhuis for reporting the issue.


  Commit: a23ba7903f7c26bf5c095cd8df5af464e2e5ccb3
      https://github.com/OpenSIPS/opensips/commit/a23ba7903f7c26bf5c095cd8df5af464e2e5ccb3
  Author: Liviu Chircu <liviu at opensips.org>
  Date:   2025-09-16 (Tue, 16 Sep 2025)

  Changed paths:
    M db/schema/address.xml
    M scripts/mysql/permissions-create.sql
    M scripts/sqlite/permissions-create.sql

  Log Message:
  -----------
  permissions: Small bugfix in DB schema

It seems bitmask values of 128 could not be stored in the table (e.g.
IPv6 addresses, not subnets), as TINYINT (char) can only support the
[-128, 127) interval.

As a fix, we silently bump the column to SMALLINT (2 bytes), solving the
issue but also not causing any breakage when pulling new packages
(e.g. no table version update, so we can backport this).


Compare: https://github.com/OpenSIPS/opensips/compare/4bf044ab5fe8...a23ba7903f7c

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