Skip to main content

Escaping quotes - moving from PHP4 to PHP5



I've inherited a php4 site that needs to run on my PHP5 Server, I've solved most of the issues but can't figure out what the author was trying to do here. Well, to be precise, he was tring to quote the submitted text but I'm not sure how this function is supposed to work and how I should do it in PHP5?







# Function to safely add slashes when magic quotes is switched off



function safe_slash($string)

{

if (!get_magic_quotes_gpc())

{

$string = addslashes($string);

}



return $string;

}




Comments

  1. By default PHP4 has an option in PHP.ini turned on called magic_quotes_gpc, it will addslashes to all $_POST/$_GET variables.

    That code simply checks if the value magic_quotes_gpc is turned off, if it is it will addslashes to the $string passed in.

    It should work in PHP4 and PHP5 (in PHP6 magic_quotes_gpc is going to be removed I believe). It's not recommended to rely on though, it was initially for 'protecting' against SQL injection but it has been found to be inadequate.

    ReplyDelete
  2. $_POST = self::addSlashesRecursive($_POST);
    $_GET = self::addSlashesRecursive($_GET);
    $_COOKIE = self::addSlashesRecursive($_COOKIE);

    function addSlashesRecursive($s)
    {
    if (get_magic_quotes_gpc()) {
    return $s;
    }
    if (is_string($s)) {
    return addslashes($s);
    } else if (is_array($s)) {
    return array_map(array('addSlashesRecursive'), $s);
    }
    return $s;
    }


    But for my mind it will be better to change your code. In PHP6 magic_quotes will be removed at all.

    ReplyDelete

Post a Comment

Popular posts from this blog

[韓日関係] 首相含む大幅な内閣改造の可能性…早ければ来月10日ごろ=韓国

div not scrolling properly with slimScroll plugin

I am using the slimScroll plugin for jQuery by Piotr Rochala Which is a great plugin for nice scrollbars on most browsers but I am stuck because I am using it for a chat box and whenever the user appends new text to the boxit does scroll using the .scrollTop() method however the plugin's scrollbar doesnt scroll with it and when the user wants to look though the chat history it will start scrolling from near the top. I have made a quick demo of my situation http://jsfiddle.net/DY9CT/2/ Does anyone know how to solve this problem?

Why does this javascript based printing cause Safari to refresh the page?

The page I am working on has a javascript function executed to print parts of the page. For some reason, printing in Safari, causes the window to somehow update. I say somehow, because it does not really refresh as in reload the page, but rather it starts the "rendering" of the page from start, i.e. scroll to top, flash animations start from 0, and so forth. The effect is reproduced by this fiddle: http://jsfiddle.net/fYmnB/ Clicking the print button and finishing or cancelling a print in Safari causes the screen to "go white" for a sec, which in my real website manifests itself as something "like" a reload. While running print button with, let's say, Firefox, just opens and closes the print dialogue without affecting the fiddle page in any way. Is there something with my way of calling the browsers print method that causes this, or how can it be explained - and preferably, avoided? P.S.: On my real site the same occurs with Chrome. In the ex