Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:76627 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 92889 invoked from network); 18 Aug 2014 05:36:14 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 18 Aug 2014 05:36:14 -0000 Authentication-Results: pb1.pair.com smtp.mail=dmitry@zend.com; spf=pass; sender-id=pass Authentication-Results: pb1.pair.com header.from=dmitry@zend.com; sender-id=pass Received-SPF: pass (pb1.pair.com: domain zend.com designates 209.85.220.182 as permitted sender) X-PHP-List-Original-Sender: dmitry@zend.com X-Host-Fingerprint: 209.85.220.182 mail-vc0-f182.google.com Received: from [209.85.220.182] ([209.85.220.182:61786] helo=mail-vc0-f182.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id F5/40-20730-AC091F35 for ; Mon, 18 Aug 2014 01:36:12 -0400 Received: by mail-vc0-f182.google.com with SMTP id hy4so5292548vcb.13 for ; Sun, 17 Aug 2014 22:36:07 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc:content-type; bh=k/t4PPMJNeaDAuepFVR7Fzn4H2vnpeag+FpmPSZRMsg=; b=Z8EiWDTsr2ILIwYNVMnLlxvTnjFejCG189noX01O8xPASDlicqrWrT/IusiW6CAPal APyoM7y99i1TIzE9xE3mJsYXdDf4CEz/3NqHsWrDuHBVsvSjX+v5etW32t++ya1io3p8 KkpkuI8LxrS5aPrjG4afztjStVV2VoYrSHHTEGT38OWPBYDiC8gTGPmhX+K+eh9SQlpq nUm4UkYDFypRtT7KsJ+hk162ej1v5zsNBBdLenRP7qxc4GvE35sQHsDCj6Zhbavkq2XF tC3tKcFsU6uyeSHTiitRcStxUgCQR+RQbQq9+FGuKmHlhUiVt3zWGDtsDigpqYVtyniu Z59w== X-Gm-Message-State: ALoCoQlK5vqLvPL9yx+k2NzJzQNSAuQkLbNAHHEiYCUx63CJNBwvPUF/ceEEdGLSruKZlePV1vP9EckMx8mMmdGX5OGafpKB1q0SJPHn4orGj85UsztoMBnbIRGFQ5qM/K+0lcckaPLw MIME-Version: 1.0 X-Received: by 10.52.253.39 with SMTP id zx7mr10420499vdc.2.1408340167478; Sun, 17 Aug 2014 22:36:07 -0700 (PDT) Received: by 10.52.110.170 with HTTP; Sun, 17 Aug 2014 22:36:07 -0700 (PDT) In-Reply-To: <327540B4-FF52-4F60-ABFF-255B8B514C14@ajf.me> References: <327540B4-FF52-4F60-ABFF-255B8B514C14@ajf.me> Date: Mon, 18 Aug 2014 09:36:07 +0400 Message-ID: To: Andrea Faulds Cc: PHP internals Content-Type: multipart/alternative; boundary=001a1135f63c67e56e0500e0bf36 Subject: Re: [PHP-DEV] phpng parameter parsing bug From: dmitry@zend.com (Dmitry Stogov) --001a1135f63c67e56e0500e0bf36 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Thanks for catching this. I'm fixing it. Dmitry. On Mon, Aug 18, 2014 at 5:57 AM, Andrea Faulds wrote: > Hi! > > In _z_param_long in Zend/zend_API.h: > > if (EXPECTED(Z_TYPE_P(arg) =3D=3D IS_LONG)) { > if (strict && UNEXPECTED(Z_DVAL_P(arg) > LONG_MAX)) { > *dest =3D LONG_MAX; > } else if (strict && UNEXPECTED(Z_DVAL_P(arg) < LONG_MIN)= ) > { > *dest =3D LONG_MIN; > } else { > *dest =3D Z_LVAL_P(arg); > } > > Note that it=E2=80=99s interpreting a long argument as a double. I expect= this > will probably go horribly, horribly wrong. > > I don=E2=80=99t have /Zend karma (yet!), could this be fixed? > > Thanks! > -- > Andrea Faulds > http://ajf.me/ > > > > > > -- > PHP Internals - PHP Runtime Development Mailing List > To unsubscribe, visit: http://www.php.net/unsub.php > > --001a1135f63c67e56e0500e0bf36--