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

[Liviu Chircu]

  Branch: refs/heads/3.6
  Home:   https://github.com/OpenSIPS/opensips
  Commit: a2ec1d4cf4e01f50f676e0bbd7010181cb135786
      https://github.com/OpenSIPS/opensips/commit/a2ec1d4cf4e01f50f676e0bbd7010181cb135786
  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.

(cherry picked from commit 8b1806ec667821eab25c0d79147a8b9cf5320074)


  Commit: 3473a217ccdc2f8e4fd977b00172cc6e705ab32a
      https://github.com/OpenSIPS/opensips/commit/3473a217ccdc2f8e4fd977b00172cc6e705ab32a
  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).

(cherry picked from commit a23ba7903f7c26bf5c095cd8df5af464e2e5ccb3)


Compare: https://github.com/OpenSIPS/opensips/compare/6d52b03aa526...3473a217ccdc

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