(release/25.0) glamor: Set need_free_region immediately after pixman region allocation

[metux]

Otherwise, the region would leak when the pixmap is not in gpu memory

Signed-off-by: stefan11111 stefan11111github@gmail.com
Part-of: https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/2289
(cherry picked from commit 41b3d99)

[metux]

🤖 Automated review — generated by Starfleet ship Enterprise (model: cruiser) on behalf of @metux. Not a human review.

🤖 Automated review — generated by Starfleet ship Enterprise on behalf of @metux. Not a human review.

Summary: Moves need_free_region = TRUE assignment before the early goto fallback path in glamor_composite_rectangles(). No behavioral change — just code reorganisation so the flag is set regardless of which exit path is taken.

Assessment:

• Backport-worthiness: No - this is code reorganisation/cleanup without behavioral change. Not a bugfix, not security-critical, not backport-worthy.
• Driver-ABI impact: No - glamor internal code, no driver-visible structures or _X_EXPORT symbols.
• Correctness: Safe reorganisation; need_free_region is set before all paths that use it.

Label: bot-review-passed