Skip to content

Tests: image filter with too large PNG dimensions - #84

Open
VadimZhestikov wants to merge 1 commit into
nginx:masterfrom
VadimZhestikov:image-filter-max-pixels
Open

VadimZhestikov wants to merge 1 commit into
nginx:masterfrom
VadimZhestikov:image-filter-max-pixels

Conversation

@VadimZhestikov

@VadimZhestikov VadimZhestikov commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Regression test for nginx/nginx#1575: PNG images with a dimension that does not fit into 16 bits are rejected with 415, instead of being understated by reading only the low 16 bits of the IHDR fields.

Covers resize, rotate (which decodes on the unfixed code) and size, for both width and height, plus a 65535 boundary control that must still be processed. The rejection cases are marked TODO for versions before 1.31.6.

This replaces the earlier image_filter_max_pixels directive test; the directive was dropped in review.

Verified: all 7 pass on the fix; on the unfixed code the five above-16-bit cases fail (as TODO) and both 65535 controls pass.

@VadimZhestikov VadimZhestikov changed the title Tests: image_filter_max_pixels directive Tests: image filter with too large PNG dimensions Sep 14, 2026
@VadimZhestikov
VadimZhestikov marked this pull request as ready for review September 14, 2026 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

2 participants