Newsgroups: perl.perl5.changes
Path: csiph.com!fu-berlin.de!bofh.it!nntp.perl.org
Xref: csiph.com perl.perl5.changes:34445
Mailing-List: contact perl5-changes-help@perl.org; run by ezmlm
Received-SPF: pass (dmarc-service-78968d7585-9zbg6: domain of github.com designates 192.30.252.201 as permitted sender) client-ip=192.30.252.201; envelope-from=noreply@github.com; helo=out-18.smtp.github.com;
Date: Tue, 10 Mar 2026 16:05:13 -0700
To: perl5-changes@perl.org
Message-ID: <Perl/perl5/push/refs/heads/blead/905eb9-e75458@github.com>
Subject: [Perl/perl5] e75458: S_newSVsv_flags_NN_PVxx - ensure that xiv_u and xn...
Mime-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Original-From: Richard Leach <noreply@github.com>
Reply-To: Richard Leach <noreply@github.com>,
Approved: news@nntp.perl.org
From: perl5-changes@perl.org (Richard Leach via perl5-changes)

  Branch: refs/heads/blead
  Home:   https://github.com/Perl/perl5
  Commit: e754582d735a91352151aa27efb79f25101b7e16
      https://github.com/Perl/perl5/commit/e754582d735a91352151aa27efb79f25101b7e16
  Author: Richard Leach <richardleach@users.noreply.github.com>
  Date:   2026-03-10 (Tue, 10 Mar 2026)

  Changed paths:
    M sv.c
    M t/op/svflags.t

  Log Message:
  -----------
  S_newSVsv_flags_NN_PVxx - ensure that xiv_u and xnv_u are initialized

When added earlier in this development cycle, `S_newSVsv_flags_NN_PVxx`
would only initialize `xiv_u` and `xnv_u` struct members in PVIV, PVNV,
and PVMG types if the relevant `IOK` or `NOK` flags were set. Since the
absence of flags technically means that the contents of these struct
members are not valid for use, skipping initialization seemed like a
way of reducing unnecessary memory store operations.

GH#24242 revealed that `PL_linestr` in _toke.c_ breaks that assumption.
`PL_linestr` behaves as an IV/PV dualvar, but is only `SvPOK` not `SvIOK`.

Given that there is a reasonable chance that this is not the only code
with that kind of behaviour, this commit adds additional initialization
to the branches that would otherwise have left `xiv_u` and/or `xnv_u`
uninitialized.



To unsubscribe from these emails, change your notification settings at https://github.com/Perl/perl5/settings/notifications