Branch: refs/heads/blead
  Home:   https://github.com/Perl/perl5
  Commit: 592ea13deca02cf378326af4dea4150f24ef17c0
      
https://github.com/Perl/perl5/commit/592ea13deca02cf378326af4dea4150f24ef17c0
  Author: Paul "LeoNerd" Evans <[email protected]>
  Date:   2026-07-16 (Thu, 16 Jul 2026)

  Changed paths:
    M sv.c

  Log Message:
  -----------
  Add extra assert()s to sprintf %p-style hacks

Such asserts would have made the bug in
https://github.com/Perl/perl5/issues/24517 stand out a little more
obviously:

```
perl: sv.c:13792: Perl_sv_vcatpvfn_flags: Assertion `hv' failed.
```


  Commit: a8ae3ee5a7cddca2590b034b027048d5f20c732b
      
https://github.com/Perl/perl5/commit/a8ae3ee5a7cddca2590b034b027048d5f20c732b
  Author: Paul "LeoNerd" Evans <[email protected]>
  Date:   2026-07-16 (Thu, 16 Jul 2026)

  Changed paths:
    M class.c
    M t/lib/croak/class

  Log Message:
  -----------
  Avoid segfault when :isa(Foo) does not load a package called Foo

A small thinko in the message generation attempted to use HvNAMEf on the
stash pointer, which would segfault when NULL (e.g. because the loaded
file did not in fact provide such a package). But we know the
superclassname SV is definitely defined and nicely printable and would
print the same string anyway, so we might as well just use that and
avoid the NULL problem.

Also delete the TODO comment that I recall was wishing that HvNAMEf
existed, when in fact it already does so. In any case the comment no
longer makes sense with this code.

Fixes https://github.com/Perl/perl5/issues/24517


Compare: https://github.com/Perl/perl5/compare/e9c4dce7543a...a8ae3ee5a7cd

To unsubscribe from these emails, change your notification settings at 
https://github.com/Perl/perl5/settings/notifications

Reply via email to