Skip to content

Commit

Permalink
Fix snmp_session_error.phpt for 32bit machines (GH-17611)
Browse files Browse the repository at this point in the history
For `$retries === PHP_INT_MAX` there is no exception on 32bit machines.
To avoid splitting the tests, or even not running it on 32bit, we work
around.
  • Loading branch information
cmb69 authored Jan 29, 2025
1 parent b667939 commit e1f3249
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion ext/snmp/tests/snmp_session_error.phpt
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ try {
}
try {
new SNMP(SNMP::VERSION_1, "$hostname:$port", $community, $timeout, PHP_INT_MAX);
echo PHP_INT_SIZE, "\n"; // no exception on 32bit machines
} catch (\ValueError $e) {
echo $e->getMessage(), PHP_EOL;
}
Expand All @@ -60,7 +61,7 @@ SNMP::__construct(): Argument #2 ($hostname) remote port must be between 0 and 6
SNMP::__construct(): Argument #2 ($hostname) remote port must be between 0 and 65535
SNMP::__construct(): Argument #2 ($hostname) length must be lower than 128
SNMP::__construct(): Argument #4 ($timeout) must be between -1 and %d
SNMP::__construct(): Argument #5 ($retries) must be between -1 and %d
%r(SNMP::__construct\(\): Argument #5 \(\$retries\) must be between -1 and %d|4)%r
SNMP::__construct(): Argument #2 ($hostname) must not contain any null bytes
SNMP::__construct(): Argument #3 ($community) must not be empty
SNMP::__construct(): Argument #3 ($community) must not contain any null bytes
Expand Down

0 comments on commit e1f3249

Please sign in to comment.