Changeset 1149

Show
Ignore:
Timestamp:
05/12/08 19:45:58 (5 months ago)
Author:
rats
Message:

* Fixed #1244: Use of uninitialized value in concatenation - in file engine/ispcp-htuser-mngr

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • trunk/engine/ispcp-htuser-mngr

    r1148 r1149  
    101101                FROM 
    102102                        domain 
    103             LIMIT 0,1 
    104         "; 
     103            LIMIT 0,1"; 
    105104 
    106105        ($rs, $rdata) = doSQL($sql); 
     
    433432                my $id = $_; 
    434433 
    435                 #$sql = "select uname from htaccess_users where id = $id"; 
    436                 $sql = "SELECT uname, count(htaccess.user_id) as c FROM htaccess_users LEFT JOIN htaccess ON htaccess_users.id=htaccess.user_id WHERE htaccess_users.id=$id GROUP BY htaccess.user_id"; 
     434                $sql = " 
     435                        SELECT 
     436                                uname, count(htaccess.user_id) as c 
     437                        FROM 
     438                                htaccess_users 
     439                        LEFT JOIN 
     440                                htaccess 
     441                                ON 
     442                                htaccess_users.id=htaccess.user_id 
     443                        WHERE 
     444                                htaccess_users.id = '$id' 
     445                        GROUP BY 
     446                                htaccess.user_id"; 
    437447 
    438448                ($rs, $rdata) = doSQL($sql); 
     
    457467                my $id = $_; 
    458468 
    459                 $sql = "select count(dmn_id) from htaccess where dmn_id = $id and path='$path'"; 
     469                $sql = "SELECT count(dmn_id) FROM htaccess WHERE dmn_id = '$id' AND path = '$path'"; 
    460470 
    461471                ($rs, $rdata) = doSQL($sql); 
     
    544554                my $id = $_; 
    545555 
    546                 $sql = "select uname, upass from htaccess_users where id = $id"; 
     556                $sql = "SELECT uname, upass FROM htaccess_users WHERE id = '$id'"; 
    547557 
    548558                ($rs, $rdata) = doSQL($sql); 
     
    588598        push_el(\@main::el, 'htuser_add_user_to_group_data()', 'Starting...'); 
    589599 
    590         my $sql = "select uname,upass from htaccess_users where id = $uid and dmn_id = $dmn_id"; 
     600        my $sql = "SELECT uname,upass FROM htaccess_users WHERE id = '$uid' AND dmn_id = '$dmn_id'"; 
    591601 
    592602        ($rs, $rdata) = doSQL($sql); 
     
    663673                my $group_id = $_; 
    664674 
    665                 $sql = <<SQL_QUERY; 
    666                         select 
     675                $sql = " 
     676                        SELECT 
    667677                                ugroup, members 
    668                         from 
     678                        FROM 
    669679                                htaccess_groups 
    670                         where 
    671                                 id = $group_id 
    672                         and 
    673                                 dmn_id = $dmn_id 
    674 SQL_QUERY 
     680                        WHERE 
     681                                id = '$group_id' 
     682                        AND 
     683                                dmn_id = '$dmn_id'"; 
    675684 
    676685                ($rs, $rdata) = doSQL($sql); 
     
    759768                my $group_id = $_; 
    760769 
    761                 $sql = "select ugroup, members, count(htaccess.id) as c from htaccess_groups left join htaccess on htaccess.group_id=htaccess_groups.id where htaccess_groups.id = $group_id and htaccess_groups.dmn_id = $dmn_id group by htaccess.group_id"; 
     770                $sql = " 
     771                SELECT 
     772                        ugroup, members, COUNT(htaccess.id) AS c 
     773                FROM 
     774                        htaccess_groups 
     775                LEFT JOIN 
     776                        htaccess ON htaccess.group_id = htaccess_groups.id 
     777                WHERE 
     778                        htaccess_groups.id = '$group_id' 
     779                AND 
     780                        htaccess_groups.dmn_id = '$dmn_id' 
     781                GROUP BY 
     782                        htaccess.group_id"; 
    762783 
    763784                ($rs, $rdata) = doSQL($sql); 
     
    775796                                my ($uname, $ucounter) = (undef, undef); 
    776797 
    777                                 $sql = "select uname from htaccess_users where id = $id"; 
     798                                $sql = "SELECT uname FROM htaccess_users WHERE id = '$id'"; 
    778799 
    779800                                ($rs, $rdata) = doSQL($sql); 
     
    789810                                return $rs if ($rs != 0); 
    790811 
    791                                 $sql = "SELECT uname, count(htaccess.user_id) as c FROM htaccess_users LEFT JOIN htaccess ON htaccess_users.id=htaccess.user_id WHERE htaccess_users.id=$id GROUP BY htaccess.user_id"; 
     812                                $sql = " 
     813                                SELECT 
     814                                        uname, COUNT(htaccess.user_id) AS c 
     815                                FROM 
     816                                        htaccess_users 
     817                                LEFT JOIN 
     818                                        htaccess ON htaccess_users.id = htaccess.user_id 
     819                                WHERE 
     820                                        htaccess_users.id = '$id' 
     821                                GROUP BY 
     822                                        htaccess.user_id"; 
    792823 
    793824                                ($rs, $rdata) = doSQL($sql); 
     
    813844                        my $id = $_; 
    814845 
    815                         $sql = "select count(dmn_id) from htaccess where dmn_id = $id and path='$path'"; 
     846                        $sql = "SELECT COUNT(dmn_id) FROM htaccess WHERE dmn_id = '$id' AND path = '$path'"; 
    816847 
    817848                        ($rs, $rdata) = doSQL($sql); 
     
    862893        push_el(\@main::el, 'htuser_mngr_engine()', 'Starting...'); 
    863894 
    864         my $sql = undef; 
    865  
    866895        # 
    867896        # user management; 
    868897        # 
    869898 
    870         $sql = <<SQL_QUERY; 
    871                 select 
     899        my $id = $main::htuser_task_id; 
     900        my $sql = " 
     901                SELECT 
    872902                        t1.dmn_id, 
    873903                        t2.domain_name, 
     
    879909                        t2.domain_uid, 
    880910                        t2.domain_gid 
    881                 from 
     911                FROM 
    882912                        htaccess as t1, 
    883913                        domain as t2 
    884                 where 
    885                         t1.id = $main::htuser_task_id 
    886                 and 
     914                WHERE 
     915                        t1.id = '$id' 
     916                AND 
    887917                        t1.user_id != 0 
    888                 and 
    889                         t1.dmn_id = t2.domain_id 
    890 SQL_QUERY 
     918                AND 
     919                        t1.dmn_id = t2.domain_id"; 
    891920 
    892921        ($rs, $rows) = doSQL($sql); 
     
    900929                # 
    901930 
    902                 $sql = <<SQL_QUERY
    903  
    904                 select 
     931                $id = $main::htuser_task_id
     932                $sql = " 
     933                SELECT 
    905934                        t1.dmn_id, 
    906935                        t2.domain_name, 
     
    912941                        t2.domain_uid, 
    913942                        t2.domain_gid 
    914                 from 
     943                FROM 
    915944                        htaccess as t1, 
    916945                        domain as t2 
    917                 where 
    918                         t1.id = $main::htuser_task_id 
    919                 and 
     946                WHERE 
     947                        t1.id = '$id' 
     948                AND 
    920949                        t1.group_id != 0 
    921                 and 
    922                         t1.dmn_id = t2.domain_id 
    923  
    924 SQL_QUERY 
     950                AND 
     951                        t1.dmn_id = t2.domain_id"; 
    925952 
    926953                ($rs, $rows) = doSQL($sql); 
     
    952979                                sys_command_rs("touch $htaccess_group_cf"); 
    953980 
    954                                 my $htaccess_users_file_name = 
    955  
    956                                 $main::cfg{'HTACCESS_USERS_FILE_NAME'}; 
    957  
    958                                 my $htaccess_groups_file_name = 
    959  
    960                                 $main::cfg{'HTACCESS_GROUPS_FILE_NAME'}; 
    961  
    962                                 my $apache_www_dir = 
    963  
    964                                 $main::cfg{'APACHE_WWW_DIR'}; 
    965  
    966                                 my $htaccess_users_file = 
    967  
    968                                 "$apache_www_dir/$dmn_name/$htaccess_users_file_name"; 
    969  
    970                                 my $htaccess_groups_file = 
    971  
    972                                 "$apache_www_dir/$dmn_name/$htaccess_groups_file_name"; 
    973  
    974                                 my $htaccess_file = 
    975  
    976                                 "$apache_www_dir/$dmn_name/$path/.htaccess"; 
     981                                my $htaccess_users_file_name = $main::cfg{'HTACCESS_USERS_FILE_NAME'}; 
     982 
     983                                my $htaccess_groups_file_name = $main::cfg{'HTACCESS_GROUPS_FILE_NAME'}; 
     984 
     985                                my $apache_www_dir = $main::cfg{'APACHE_WWW_DIR'}; 
     986 
     987                                my $htaccess_users_file = "$apache_www_dir/$dmn_name/$htaccess_users_file_name"; 
     988 
     989                                my $htaccess_groups_file = "$apache_www_dir/$dmn_name/$htaccess_groups_file_name"; 
     990 
     991                                my $htaccess_file =     "$apache_www_dir/$dmn_name/$path/.htaccess"; 
    977992 
    978993                        } 
     
    984999                                if ($rs == 0) { 
    9851000 
    986                                         $sql = "update htaccess set status='ok' where id = $htuser_id"; 
     1001                                        $sql = "UPDATE htaccess SET status = 'ok' WHERE id = $htuser_id"; 
    9871002 
    9881003                                } else { 
     
    9921007                                        $msg =~ s/\'/\\\'/g; 
    9931008 
    994                                         $sql = "update htaccess set status='$sub_name | $msg' where id = $htuser_id;"; 
     1009                                        $sql = "UPDATE htaccess SET status= '$sub_name | $msg' WHERE id = '$htuser_id';"; 
    9951010 
    9961011                                } 
     
    10061021                                if ($rs == 0) { 
    10071022 
    1008                                         $sql = "delete from htaccess where id = $htuser_id"; 
     1023                                        $sql = "DELETE FROM htaccess WHERE id = '$htuser_id'"; 
    10091024 
    10101025                                } else { 
     
    10141029                                        $msg =~ s/\'/\\\'/g; 
    10151030 
    1016                                         $sql = "update htaccess set status='$sub_name | $msg' where id = $htuser_id;"; 
     1031                                        $sql = "UPDATE htaccess SET status = '$sub_name | $msg' WHERE id = $htuser_id;"; 
    10171032 
    10181033                                } 
     
    10851100                        if ($rs == 0) { 
    10861101 
    1087                                 $sql = "update htaccess set status='ok' where id = $htuser_id"; 
     1102                                $sql = "UPDATE htaccess SET status = 'ok' WHERE id = '$htuser_id';"; 
    10881103 
    10891104                        } else { 
     
    10931108                                $msg =~ s/\'/\\\'/g; 
    10941109 
    1095                                 $sql = "update htaccess set status='$sub_name | $msg' where id = $htuser_id;"; 
     1110                                $sql = "UPDATE htaccess SET status = '$sub_name | $msg' WHERE id = '$htuser_id';"; 
    10961111 
    10971112                        } 
     
    11071122                        if ($rs == 0) { 
    11081123 
    1109                                 $sql = "delete from htaccess where id = $htuser_id"; 
     1124                                $sql = "DELETE FROM htaccess WHERE id = '$htuser_id';"; 
    11101125 
    11111126                        } else { 
     
    11151130                                $msg =~ s/\'/\\\'/g; 
    11161131 
    1117                                 $sql = "update htaccess set status='$sub_name | $msg' where id = $htuser_id;"; 
     1132                                $sql = "UPDATE htaccess SET status = '$sub_name | $msg' WHERE id = '$htuser_id';"; 
    11181133 
    11191134                        } 
     
    11351150                        my $htaccess_data = @$rdata[0]; 
    11361151 
    1137                         $sql = "select uname, upass, dmn_id, status, id from htaccess_users where dmn_id = '".@$htaccess_data[0]."' and status != 'ok' "; 
     1152                        $sql = " 
     1153                        SELECT 
     1154                                uname, upass, dmn_id, status, id 
     1155                        FROM 
     1156                                htaccess_users 
     1157                        WHERE 
     1158                                dmn_id = '".@$htaccess_data[0]."' 
     1159                        AND 
     1160                                status != 'ok';"; 
    11381161 
    11391162                                ($rs, $rows) = doSQL($sql); 
     
    11671190                                                                return $rs if ($rs != 0); 
    11681191 
    1169                                                                 $sql = "update htaccess_users set status='ok' where id = @$query_user[4]"; 
     1192                                                                $sql = "UPDATE htaccess_users SET status = 'ok' WHERE id = '".@$query_user[4]."';"; 
    11701193 
    11711194                                                } elsif ($htpasswd_status eq 'delete') { 
     
    11761199                                                                return $rs if ($rs != 0); 
    11771200 
    1178                                                                 $sql = "delete from htaccess_users where id = @$query_user[4]"; 
     1201                                                                $sql = "DELETE FROM htaccess_users WHERE id = '".@$query_user[4]."';"; 
    11791202 
    11801203                                                } elsif ($htpasswd_status eq 'toadd') { 
     
    11851208                                                                return $rs if ($rs != 0); 
    11861209 
    1187                                                                 $sql = "update htaccess_users set status='ok' where id = @$query_user[4]"; 
     1210                                                                $sql = "UPDATE htaccess_users SET status = 'ok' WHERE id = '".@$query_user[4]."';"; 
    11881211 
    11891212                                                } elsif ($htpasswd_status eq 'ok') { 
    1190  
     1213                                                                ## TODO: ? 
    11911214 
    11921215                                                } else { 
     
    11941217                                                                push_el(\@main::el, 'htuser-usermanagement()', 'Failure modifying user in Domain '.$htpasswd_dnm.' : '.$htpasswd_name); 
    11951218 
    1196                                                                 $sql = "update htaccess_users set status='failure' where id =  @$query_user[4]"; 
     1219                                                                $sql = "UPDATE htaccess_users SET status='failure' WHERE id = '".@$query_user[4]."';"; 
    11971220 
    11981221                                                }