FAQ
Hi all,

pg_unescape_bytea() should raise error for invalid inputs
https://bugs.php.net/bug.php?id=65165

We need to discuss if E_WARING is proper, which branch should be
changed.

It will not break working code, so I would like to use E_WARNING and
change it from PHP 5.4.

Any comments?

Regards,

--
Yasuo Ohgaki
yohgaki@ohgaki.net

Search Discussions

  • Matteo Beccati at Jul 2, 2013 at 7:57 am
    Hi Yasuo,
    pg_unescape_bytea() should raise error for invalid inputs
    https://bugs.php.net/bug.php?id=65165

    We need to discuss if E_WARING is proper, which branch should be
    changed.

    It will not break working code, so I would like to use E_WARNING and
    change it from PHP 5.4.
    Raising an E_WARNING will most likely mean that it needs to be
    suppressed in some way, which I think is not necessary.

    I'm all for returning false when the input is invalid. However, given
    the behaviour change, I suppose it would be PHP 5.next material.


    Cheers
    --
    Matteo Beccati

    Development & Consulting - http://www.beccati.com/
  • Yasuo Ohgaki at Jul 2, 2013 at 10:35 am
    Hi Matteo,

    2013/7/2 Matteo Beccati <php@beccati.com>
    Raising an E_WARNING will most likely mean that it needs to be
    suppressed in some way, which I think is not necessary.

    I'm all for returning false when the input is invalid. However, given
    the behaviour change, I suppose it would be PHP 5.next material.
    If pg_unecape_bytea() returns FALSE, it means there is obvious bug
    or some kind of attack.

    Users should be prepared for unexpected errors not only pg_escape_bytea()
    but also many other functions anyway.

    It does not hart any application, so it's good to have 5.5. IMHO.

    Regards,

    --
    Yasuo Ohgaki
    yohgaki@ohgaki.net

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
groupphp-internals @
categoriesphp
postedJun 29, '13 at 2:46a
activeJul 2, '13 at 10:35a
posts3
users2
websitephp.net

2 users in discussion

Yasuo Ohgaki: 2 posts Matteo Beccati: 1 post

People

Translate

site design / logo © 2022 Grokbase