Changeset 530

Show
Ignore:
Timestamp:
04/22/08 14:48:35 (8 months ago)
Author:
sidtheduck
Message:

+ Fix for ticket #150 - collection and album paths were being escaped twice before comparison, so mysql query was not matching paths with apostrophes (') or parenthesis"()", etc.
+ Updated all instances of mysql_escape_string() to mysql_real_escape_string() for consistency (however, now requires PHP<=4.3.0)

Location:
trunk
Files:
5 modified

Legend:

Unmodified
Added
Removed

  • trunk/admin/plog-admin-functions.php

    r522 r530  
    7171        $create_path = $dest_collection_name."/".$dest_album_name; 
    7272 
    73         while (is_file('../images/'.$create_path."/".$unique_filename_base . "." . $filename_ext)){ 
     73        while (is_file($config['basedir'].'images/'.$create_path."/".$unique_filename_base . "." . $filename_ext)){ 
    7474                $unique_filename_base = $filename_base . " (" . ++$i .")"; 
    7575        } 
    … …  
    133133                VALUES 
    134134          ('".$albumdata['collection_id']."', 
    135            '".$albumdata['album_id']."','".mysql_escape_string($picture_path)."', 
     135           '".$albumdata['album_id']."','".mysql_real_escape_string($picture_path)."', 
    136136           NOW(), 
    137137           NOW(), 
    138138           1, 
    139            '".mysql_escape_string($exif["date_taken"])."', 
    140            '".mysql_escape_string($exif["camera"])."', 
    141            '".mysql_escape_string($exif["shutter_speed"])."', 
    142            '".mysql_escape_string($exif["focal_length"])."', 
    143            '".mysql_escape_string($exif["flash"])."', 
    144            '".mysql_escape_string($exif["aperture"])."', 
    145            '".mysql_escape_string($caption)."', 
    146            '".mysql_escape_string($desc)."')"; 
     139           '".mysql_real_escape_string($exif["date_taken"])."', 
     140           '".mysql_real_escape_string($exif["camera"])."', 
     141           '".mysql_real_escape_string($exif["shutter_speed"])."', 
     142           '".mysql_real_escape_string($exif["focal_length"])."', 
     143           '".mysql_real_escape_string($exif["flash"])."', 
     144           '".mysql_real_escape_string($exif["aperture"])."', 
     145           '".mysql_real_escape_string($caption)."', 
     146           '".mysql_real_escape_string($desc)."')"; 
    147147            
    148148        $sql_result = run_query($query); 
    … …  
    348348        }; 
    349349 
    350         $target_name = strtolower(sanitize_filename($name)); 
     350        $target_name = strtolower(sanitize_filename(SmartStripSlashes($name))); 
    351351         
    352352 
    … …  
    410410                $album_path = $row['album_path']; 
    411411 
    412                 $new_path = mysql_escape_string($target_name."/".$album_path."/".$filename); 
     412                $new_path = mysql_real_escape_string($target_name."/".$album_path."/".$filename); 
    413413 
    414414                // update database 
    … …  
    511511        }; 
    512512 
    513         $album_folder = strtolower(sanitize_filename($album_name)); 
     513        $album_folder = strtolower(sanitize_filename(SmartStripSlashes($album_name))); 
    514514 
    515515        // first try to create the directory to hold the images, if that fails, then the album 
    … …  
    545545        $errors = $output = ""; 
    546546 
    547         $target_name = strtolower(sanitize_filename($name)); 
     547        $target_name = strtolower(sanitize_filename(SmartStripSlashes($name))); 
    548548 
    549549        $album_id = intval($album_id); 

  • trunk/admin/plog-options.php

    r509 r530  
    2323        $query = "UPDATE `".TABLE_PREFIX."config` SET 
    2424                `truncate`='".intval($_POST["truncate"])."', 
    25                 `feed_title`='".mysql_escape_string($_POST["feed_title"])."', 
    26                 `feed_language`='".mysql_escape_string($_POST["feed_language"])."', 
     25                `feed_title`='".mysql_real_escape_string($_POST["feed_title"])."', 
     26                `feed_language`='".mysql_real_escape_string($_POST["feed_language"])."', 
    2727                `feed_num_entries`='".intval($_POST["feed_num_entries"])."', 
    2828                `allow_dl`='".intval($allow_dl)."', 
    2929                `allow_comments`='".intval($allow_comments)."', 
    3030                `allow_print`='".intval($allow_print)."', 
    31                 `default_sortby`='".mysql_escape_string($_POST["default_sortby"])."', 
    32                 `default_sortdir`='".mysql_escape_string($_POST["default_sortdir"])."', 
    33                 `album_sortby`='".mysql_escape_string($_POST["album_sortby"])."', 
    34                 `album_sortdir`='".mysql_escape_string($_POST["album_sortdir"])."', 
    35                 `collection_sortby`='".mysql_escape_string($_POST["collection_sortby"])."', 
    36                 `collection_sortdir`='".mysql_escape_string($_POST["collection_sortdir"])."', 
     31                `default_sortby`='".mysql_real_escape_string($_POST["default_sortby"])."', 
     32                `default_sortdir`='".mysql_real_escape_string($_POST["default_sortdir"])."', 
     33                `album_sortby`='".mysql_real_escape_string($_POST["album_sortby"])."', 
     34                `album_sortdir`='".mysql_real_escape_string($_POST["album_sortdir"])."', 
     35                `collection_sortby`='".mysql_real_escape_string($_POST["collection_sortby"])."', 
     36                `collection_sortdir`='".mysql_real_escape_string($_POST["collection_sortdir"])."', 
    3737                `thumb_num`='".intval($_POST["thumb_num"])."', 
    3838                `compression`='".intval($_POST["image_quality"])."', 
    39                 `admin_username`='".mysql_escape_string($_POST["admin_username"])."', 
    40                 `admin_email`='".mysql_escape_string($_POST["admin_email"])."', 
    41                 `date_format`='".mysql_escape_string($_POST["date_format"])."', 
     39                `admin_username`='".mysql_real_escape_string($_POST["admin_username"])."', 
     40                `admin_email`='".mysql_real_escape_string($_POST["admin_email"])."', 
     41                `date_format`='".mysql_real_escape_string($_POST["date_format"])."', 
    4242                `use_mod_rewrite`='".intval(@$_POST["use_mod_rewrite"])."', 
    4343                `square_thumbs`='".intval($square_thumbs)."', 
    4444                `comments_notify`='".intval($_POST["comments_notify"])."', 
    4545                `comments_moderate`='".intval($comments_moderate)."', 
    46                 `gallery_url`='".mysql_escape_string($_POST["gallery_url"])."', 
    47                 `gallery_name`='".mysql_escape_string($_POST["gallery_name"])."', 
     46                `gallery_url`='".mysql_real_escape_string($_POST["gallery_url"])."', 
     47                `gallery_name`='".mysql_real_escape_string($_POST["gallery_name"])."', 
    4848                `thumb_nav_range`='".intval($_POST["thumb_nav_range"])."', 
    4949                `enable_thumb_nav`='".intval(@$_POST["enable_thumb_nav"])."', 

  • trunk/admin/plog-upload.php

    r518 r530  
    109109                if ($_REQUEST["destination_radio"] == "new"){ 
    110110                        // Create the new album 
    111                         $result = add_album(mysql_escape_string($_REQUEST["new_album_name"]), NULL, $_REQUEST["collections_menu"]); 
     111                        $result = add_album(mysql_real_escape_string($_REQUEST["new_album_name"]), NULL, $_REQUEST["collections_menu"]); 
    112112                        $album_id = $result["id"]; 
    113113                } else { 

  • trunk/lib/plogger/install_functions.php

    r517 r530  
    292292        $config['gallery_name'] = $form['gallery_name']; 
    293293         
    294         $config = array_map('mysql_escape_string',$config); 
     294        $config = array_map('mysql_real_escape_string',$config); 
    295295 
    296296        $query = "INSERT INTO `".TABLE_PREFIX."config` 

  • trunk/plog-functions.php

    r529 r530  
    920920        foreach($levels as $key => $level) { 
    921921                if (isset($path_parts[$key])) { 
    922                         $names[$level] = mysql_escape_string(urldecode(SmartStripSlashes($path_parts[$key]))); 
     922                        $names[$level] = mysql_real_escape_string(urldecode(SmartStripSlashes($path_parts[$key]))); 
    923923                        $current_level = $level; 
    924924                } 
    … …  
    928928                $sql = "SELECT * 
    929929                        FROM `".TABLE_PREFIX."collections` 
    930                         WHERE `path`='".mysql_real_escape_string($names["collection"])."'"; 
     930                        WHERE `path`='".$names["collection"]."'"; 
    931931                $result = run_query($sql); 
    932932                 
    … …  
    945945                $sql = "SELECT * 
    946946                        FROM `".TABLE_PREFIX."albums` 
    947                         WHERE `path`='".mysql_real_escape_string($names["album"])."' 
     947                        WHERE `path`='".$names["album"]."' 
    948948                                AND `parent_id`=".intval($collection["id"]); 
    949949                $result = run_query($sql); 
    … …  
    980980                $sql = "SELECT * 
    981981                        FROM `".TABLE_PREFIX."pictures` 
    982                         WHERE `caption`='".mysql_real_escape_string($names["picture"])."' 
     982                        WHERE `caption`='".$names["picture"]."' 
    983983                                AND `parent_album`=".intval($album["id"]); 
    984984                $result = run_query($sql); 
    … …  
    991991                        $like_match = array("_", "%"); 
    992992                        $like_replace = array("\_", "\%"); 
    993                         $esc_filepath = str_replace($like_match, $like_replace, $filepath); 
     993                        $filepath = str_replace($like_match, $like_replace, $filepath); 
    994994                        $sql = "SELECT * 
    995995                                FROM `".TABLE_PREFIX."pictures` 
    996                                 WHERE `path` LIKE '" . mysql_real_escape_string($esc_filepath)."____' 
     996                                WHERE `path` LIKE '".$filepath."____' 
    997997                                        AND `parent_album`=".intval($album["id"]); 
    998998                        $result = run_query($sql); 
    … …  
    15131513                foreach ($terms as $term) { 
    15141514                        $query .= " 
    1515                                 `path` LIKE '%".mysql_escape_string($term)."%' OR 
    1516                                 `description` LIKE '%".mysql_escape_string($term)."%' OR 
    1517                                 `comment` LIKE '%".mysql_escape_string($term)."%' OR 
    1518                                 `caption` LIKE '%".mysql_escape_string($term)."%' OR "; 
     1515                                `path` LIKE '%".mysql_real_escape_string($term)."%' OR 
     1516                                `description` LIKE '%".mysql_real_escape_string($term)."%' OR 
     1517                                `comment` LIKE '%".mysql_real_escape_string($term)."%' OR 
     1518                                `caption` LIKE '%".mysql_real_escape_string($term)."%' OR "; 
    15191519                } 
    15201520