Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:77643
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain zend.com designates 209.85.220.173 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <54242A0C.3000309@gmail.com>
References: <07153B91-E12F-4B16-ADD7-86CFC75C4642@ajf.me>
	<54220EB4.60001@sugarcrm.com>
	<6A21F4EF-476E-476C-A64B-246B8929172B@ajf.me>
	<5422DD87.80109@gmail.com>
	<568C0F03-B23B-43FC-963D-6D0D23405E9D@ajf.me>
	<54230209.2050406@sugarcrm.com>
	<54233C70.3060604@gmail.com>
	<54236D16.3070401@sugarcrm.com>
	<B1A90F30-A7A0-4C51-9EDB-29A851CA0E0F@ajf.me>
	<54242A0C.3000309@gmail.com>
Date: Fri, 26 Sep 2014 10:57:27 +0400
Message-ID: <CA+9eiLuxd-XVYC4DnXg5XnmoApBPr_9xOTcOsz+d0oSijsmDSQ@mail.gmail.com>
To: Rowan Collins <rowan.collins@gmail.com>
Cc: "internals@lists.php.net" <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=20cf30780ab01215200503f26ed1
Subject: Re: [PHP-DEV] [RFC] ZPP Failure On Overflow
From: dmitry@zend.com (Dmitry Stogov)

--20cf30780ab01215200503f26ed1
Content-Type: text/plain; charset=UTF-8

May be this overflow checks should be optional?
In old ages compilers for Wirth family languages had option to enable or
disable overflow checks.
If option is enabled we may throw an exception, if no - work as today.
Any user would be able to chose between compatibility and safety.

Thanks. Dmitry.

On Thu, Sep 25, 2014 at 6:43 PM, Rowan Collins <rowan.collins@gmail.com>
wrote:

> Andrea Faulds wrote (on 25/09/2014):
> >> No bogus value ever gets to a function - it always gets INT_MAX on
> >> > overflow. If INT_MAX is harmful for this function, this change does
> not
> >> > help as you could still pass INT_MAX and this change would not do
> anything.
> >>
> > No it won't. Normally it truncates (module), only some functions cap.
>
> I saw that in the RFC, and thought it rather odd that a more sane
> implementation already exists, but functions have to opt in to use it.
>
> I thought perhaps it was a side-effect of something else, but
> README.PARAMETER_PARSING_API doesn't list any other difference between "l"
> and "L":
>
> > l  - long (long)
> > L  - long, limits out-of-range numbers to LONG_MAX/LONG_MIN (long)
>
> What was the original motivation for functions to have that choice?
> --
> Rowan Collins
> [IMSoP]
>
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>

--20cf30780ab01215200503f26ed1--