Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Segmentation fault (stack overflow) in AppendIterator #15911

Open
YuanchengJiang opened this issue Sep 16, 2024 · 5 comments
Open

Segmentation fault (stack overflow) in AppendIterator #15911

YuanchengJiang opened this issue Sep 16, 2024 · 5 comments
Labels
Bug Extension: spl Status: Verified

Comments

@YuanchengJiang
Copy link

Description

The following code:

<?php
$iterator = new \AppendIterator();
$iterator->append($iterator);

PHP Version

PHP 8.4.0-dev

Operating System

ubuntu 22.04
@devnexen
Copy link
Member

seems spl issue, might be wrong.

@cmb69
Copy link
Member

cmb69 commented Sep 16, 2024

Well, that's infinite recursion. See #9104.

@nielsdos
Copy link
Member

Related to #15672

DanielEScherzer added a commit to DanielEScherzer/php-src that referenced this issue Oct 18, 2024
@DanielEScherzer
phpGH-15911: prevent appending an AppendIterator to itself
9e348d4
DanielEScherzer added a commit to DanielEScherzer/php-src that referenced this issue Oct 18, 2024
@DanielEScherzer
phpGH-15911: prevent appending an AppendIterator to itself
f88ffd5
DanielEScherzer added a commit to DanielEScherzer/php-src that referenced this issue Oct 18, 2024
@DanielEScherzer
phpGH-15911: prevent appending an AppendIterator to itself
d5933b5
DanielEScherzer added a commit to DanielEScherzer/php-src that referenced this issue Oct 18, 2024
@DanielEScherzer
phpGH-15911: prevent appending an AppendIterator to itself
78a5e02
@DanielEScherzer
Copy link
Contributor

DanielEScherzer commented Oct 18, 2024 •
edited
Loading

Per https://3v4l.org/Dvcb0 this has been around 8.1 (or earlier), patch targeting 8.2 sent at #16492

@Girgias
Copy link
Member

Girgias commented Oct 20, 2024

I don't think there is anything meaningful that can be done here. Infinite recursion is a programming error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Extension: spl Status: Verified
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@cmb69 @devnexen @nielsdos @Girgias @YuanchengJiang @DanielEScherzer