Project

General

Profile

Actions

Bug #14475

closed

PHP Error: suricata_check_for_rule_updates.php:820

Added by Steve Wheeler about 2 years ago. Updated about 2 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
Suricata
Target version:
-
Start date:
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Affected Version:
Affected Plus Version:
23.05
Affected Architecture:

Description

PHP Error from Suricata when updating:

Crash report begins.  Anonymous machine information:

amd64
14.0-CURRENT
FreeBSD 14.0-CURRENT #1 plus-RELENG_23_05-n256102-7cd3d043045: Mon May 22 15:33:52 UTC 2023     root@freebsd:/var/jenkins/workspace/pfSense-Plus-snapshots-23_05-main/obj/amd64/LkEyii3W/var/jenkins/workspace/pfSense-Plus-snapshots-23_05-main/sources/FreeBS

Crash report details:

PHP Errors:
[14-Jun-2023 01:00:58 Europe/Berlin] PHP Fatal error:  Uncaught ValueError: gettext(): Argument #1 ($message) is too long in /usr/local/pkg/suricata/suricata_check_for_rule_updates.php:820
Stack trace:
#0 /usr/local/pkg/suricata/suricata_check_for_rule_updates.php(820): gettext('- Snort rules: ...')
#1 {main}
  thrown in /usr/local/pkg/suricata/suricata_check_for_rule_updates.php on line 820

No FreeBSD crash data found.

See: https://forum.netgate.com/topic/180826/23-05-suricata-update-pfsense-php-crash

Actions #1

Updated by Bill Meeks about 2 years ago

Not sure exactly why the input string is too long in this case, but I did find in the PHP interpreter source code that the maximum length for an input string to the gettext() function is 4096 characters.

I will add a limit of 4096 characters to the input string for the gettext() call in that section of code. I will include this in the next Suricata GUI package update.

Actions #2

Updated by Bill Meeks about 2 years ago

A fix for this has been submitted in Pull Request 1271: https://github.com/pfsense/FreeBSD-ports/pull/1271. This issue can be marked resolved when the pull request is merged.

Actions #3

Updated by Jim Pingle about 2 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

PR Merged

Actions

Also available in: Atom PDF