Поиск и исправление багов в исходниках PHP

[mayorovp]

В коде патча ничего не понял, но предполагаю что он завалится на вот этом тесте:

class Foo implements JsonSerializable {
  public function jsonSerialize() {
    return new Bar;
  }
}
class Bar implements JsonSerializable {
  public function jsonSerialize() {
    return new Foo;
  }
}

[aivus]

Тоже самое написали и в PR:
https://github.com/php/php-src/pull/2763#discussion_r143798219

[muhammad_97]

Или вот таком:

class Foo implements JsonSerializable
{
    public function jsonSerialize() {
        return [new self];
    }
}

[mayorovp]

Или даже вот таком:

class Foo implements JsonSerializable
{
    public function jsonSerialize() {
        return [this];
    }
}

[akadex]

Есть вполне понятный debug json, когда он возвращает ошибку. Она имеет документацию http://php.net/manual/ru/function.json-last-error.php

[samizdam]

Не понял как ваш комментарий относится к теме статьи…
Segfault же.…

[Viacheslav01]

Потер, не гоже.

[kenbo]

В комментарии к PR совершенно верно заявили что это не баг а фича, главная же проблема это переполнение стека.

[villav]

Chicago PHP UG represent!

[4ainik]

Основной баг как раз в вашем рнр коде, как вы сами и написали рекурсивный вызов, который и так очевиден, без анализа исходников.
По хорошему метод jsonSerialize должен возвращать строку, а не новый экземпляр какого-то объекта и не массив как тут некоторые предлагают :)
Забавно, что на винхп на рнр версии 5.2.4 нет никаких segfault, а на экране печатается

string(2) "{}"

Правда класс объявлен без

implements JsonSerializable

т.к. моя сбока рнр не знает о существовании JsonSerializable интерфейса.
Но в целом вы получили разносторонний опыт…

[bolk]

JsonSerializable появился в 5.4.
> Забавно, что на винхп на рнр версии 5.2.4 нет никаких segfault, а на экране печатается
Да ничего забавного. Так вы не реализуете интерфейс, указанная функция просто не вызывается.