Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:103766
MIME-Version: 1.0
References: <CADqTB_gq1cj44ndjODNTPCHTJ8ZQS4DtJE=gyG5DG8oqJhSb0A@mail.gmail.com>
 <b1499ac754a80462459f6d72a8b8252d@gmail.com>
In-Reply-To: <b1499ac754a80462459f6d72a8b8252d@gmail.com>
Date: Fri, 18 Jan 2019 12:51:55 +0100
Message-ID: <CAF+90c8XEWHovw+BzLm-JxUa9oWUCmCOo0_M3o362DQgS5WhRg@mail.gmail.com>
To: =?UTF-8?Q?Lauri_Kentt=C3=A4?= <lauri.kentta@gmail.com>
Cc: Rasmus Schultz <rasmus@mindplay.dk>, PHP internals <internals@lists.php.net>, 
	Nikita Popov <nikic@php.net>
Content-Type: multipart/alternative; boundary="0000000000002b5354057fba24e0"
Subject: Re: [PHP-DEV] Possible bug in the implementation of php://input streams?
From: nikita.ppv@gmail.com (Nikita Popov)

--0000000000002b5354057fba24e0
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Wed, Jan 16, 2019 at 9:16 PM Lauri Kentt=C3=A4 <lauri.kentta@gmail.com> =
wrote:

> On 2019-01-16 09:59, Rasmus Schultz wrote:
> > We've noticed something odd about the "php://input" stream.
> >
> > If you attempt to rewind() it after reading the stream, rewind()
> > returns
> > true, and ftell() subequently returns 0.
> >
> > However, attempting to read the stream again after that returns
> > nothing.
>
> It has so many layers of redirection that someone missed setting
> stream->position on one layer.
>
> Maybe someone would care to apply the attached patch to fix this.
>

Thanks Lauri, I have applied your patch to PHP 7.2 and up.

Nikita

--0000000000002b5354057fba24e0--