Skip to content

fix(IpNormalizer): Added inet_pton() check before bitwise operations - #57127

Merged
susnux merged 1 commit into
masterfrom
addCheckBeforeBitwise
Dec 17, 2025
Merged

fix(IpNormalizer): Added inet_pton() check before bitwise operations#57127
susnux merged 1 commit into
masterfrom
addCheckBeforeBitwise

Conversation

@solracsf

@solracsfsolracsf commented Dec 16, 2025

Copy link
Copy Markdown
Member

Summary

Added explicit check for inet_pton() returning false before bitwise operations to prevent potential issues with invalid IPs + small optimizations.

Checklist

@solracsfsolracsf added this to the Nextcloud 33 milestone Dec 16, 2025
@solracsf
solracsf requested a review from a team as a code ownerDecember 16, 2025 14:01
@solracsfsolracsf added bug 3. to review Waiting for reviews labels Dec 16, 2025
@solracsf
solracsf requested review from Altahrim, CarlSchwan, icewind1991 and salmart-dev and removed request for a teamDecember 16, 2025 14:01
@solracsf
solracsfforce-pushed the addCheckBeforeBitwise branch from 5dbb7a2 to b9d55baCompareDecember 16, 2025 14:06
Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
@susnux
susnux merged commit cc23025 into masterDec 17, 2025
199 of 201 checks passed
@susnux
susnux deleted the addCheckBeforeBitwise branch December 17, 2025 15:28
@susnux

Copy link
Copy Markdown
Contributor

/backport to stable32

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: PHP warning: Unsupported operand types: bool & string

3 participants

@solracsf@susnux@Altahrim