同時に試してキャッチを実行する

Question

だから私はこのOOPログイン&ユーザーが彼/彼女の名前を変更する機能を与える登録システムを持っています。

ユーザーが「更新」ボタンを押すと、DB内の名前が変更されます。問題は、リダイレクトするユーザーをリダイレクトするのではなく、DB内の名前を変更することです。その名前を変更できなかった場合にのみ表示されるエラーです。

User.php

<?php 

class User{ 
private $_db, 
     $_data, 
     $_sessionName, 
     $_cookieName, 
     $_isLoggedIn; 

public function __construct($user = null){ 
    $this->_db = DB::getInstance(); 

    $this->_sessionName = Config::get('session/session_name'); 
    $this->_cookieName = Config::get('remember/cookie_name'); 

    if(!$user){ 
     if(Session::exists($this->_sessionName)){ 
      $user = Session::get($this->_sessionName); 

      if($this->find($user)){ 
       $this->_isLoggedIn = true; 
      } else{ 
       //process logout 
      } 
     } 
    } else{ 
     $this->find($user); 
    } 
} 

public function update($fields = array(), $id = null){ 

    if(!$id && $this->isLoggedIn()){ 
     $id = $this->data()->id; 
    } 

    if(!$this->_db->update('users', $id, $fields)){ 
     throw new Exception('There was a problem updating your profile.'); 
    } 
} 

public function create($fields = array()){ 
    if(!$this->_db->insert('users', $fields)){ 
     throw new Exception('There was a problem creating an account'); 
    } 
} 

public function find($user = null){ 
    if($user){ 
     $field = (is_numeric($user)) ? 'id' : 'username'; 
     $data = $this->_db->get('users', array($field, '=', $user)); 

     if($data->count()){ 
      $this->_data = $data->first(); 
      return true; 
     } 
    } 
    return false; 
} 

public function login($username = null, $password = null, $remember = false){ 
    if(!$username && !$password && $this->exists()){ 
     Session::put($this->_sessionName, $this->data()->id); 
    } else{ 
     $user = $this->find($username); 
     if($user){ 
      if($this->data()->password === Hash::make($password, $this->data()->salt)){ 
       Session::put($this->_sessionName, $this->data()->id); 

       if($remember){ 
        $hash = Hash::unique(); 
        $hashCheck = $this->_db->get('users_session', array('user_id', '=', $this->data()->id)); 

        if(!$hashCheck->count()){ 
         $this->_db->insert('users_session',array(
          'user_id' => $this->data()->id, 
          'hash' => $hash 
         )); 
        } else{ 
         $hash = $hashCheck->first()->hash; 
        } 
        Cookie::put($this->_cookieName, $hash, Config::get('remember/cookie_expiry')); 
       } 

       return true; 
      } 
     } 
    } 
    return false; 
} 

public function exists(){ 
    return(!empty($this->_data)) ? true : false; 
} 

public function logout(){ 
    $this->_db->delete('users_session', array('user_id', '=', $this->data()->id)); 

    Session::delete($this->_sessionName); 
    Cookie::delete(Config::get('remember/cookie_name')); 
} 

public function data(){ 
    return $this->_data; 
} 

public function isLoggedIn(){ 
    return $this->_isLoggedIn; 
} 
} 

<?php 

    $user = new User(); 
    if(!$user->isLoggedIn()){ 
     Redirect::to('login'); 
    } 

    if(Input::exists()){ 
     if(Token::check(Input::get('token'))){ 

      $validate = new Validate(); 
      $validation = $validate->check($_POST, array(
       'name' => array(
        'required' => true, 
        'min' => 2, 
        'max' => 50 
       ) 
      )); 

      if($validation->passed()){ 

       try{ 
        $user->update(array(
         'name' => Input::get('name') 
        )); 

        Session::flash('flash', 'Your profile has been edited with success!'); 
        Redirect::to('flash'); 

       } catch(Exception $e){ 
        die($e->getMessage()); 
       } 
      } else{ 
       foreach($validation->errors() as $error){ 
        echo $error . '<br />'; 
       } 
      } 

     } 
    } 

    ?> 

    <form action="" method="post"> 
     <div class="field"> 
      <input type="text" name="name" value="<?php echo escape($user->data()->name); ?>"> 
     </div> 
     <input type="submit" value="Update"> 
     <input type="hidden" name="token" value="<?php echo Token::generate(); ?>"> 
    </form> 

私は

これが起こっている理由は見当もつかないが、update（）メソッド内で編集profile.php私のDB.class

代わりに

public function update($table, $id, $fields){ 

    $set = []; 

    foreach ($fields as $name => $value) { 
     $set[] = "{$name} = ?"; 
    } 

    $set = implode(', ', $set); 

    $sql = "UPDATE {$table} SET {$set} WHERE id = ?"; 
    $fields[] = $id; //id should always be the last ? 
    if($this->query($sql, $fields)->error()){ 
     return true; 
    } 
    return false; 
} 

を行い、あなたの更新機能

public function update($table, $id, $fields){ 
    $set = ''; 
    $x = 1; 
    foreach ($fields as $name => $value) { 
     $set .= "{$name} = ?"; 
     if($x < count($fields)){ 
      $set .= ', '; 
     } 
     $x++; 
    } 
    $sql = "UPDATE {$table} SET {$set} WHERE id = {$id}"; 
    if($this->query($sql, $fields)->error()){ 
     return true; 
    } 
    return false; 
} 

については

Answer 1

はまた、SQLインジェクションのためのベクターとして使用するtableとidと$fieldsのも、鍵の可能性があります。これまでに使用した場所に入力している可能性があります。しかし、ホワイトリストと比較してテーブルをチェックしていないため、SQLに値を連結するたびに、それらが悪用される可能性があります。それが取るのは間違い1つですが、他の場所でのコーディングエラーでセキュリティが損なわれることは許されません。

データベース自体からテーブルのリストを取得できます。

$statement = 'SELECT `TABLE_NAME` FROM `information_schema`.`TABLES` WHERE `TABLE_SCHEMA` LIKE "'.$database.'"'; 

だから、あなたがDBに接続するときには、許容可能なテーブルのリストを設定することができ、テーブルが（in_arrayかなと）に置かれたときに確認してください。ちょっとした考え。

$fields配列のキーを含むこともできます。これは、表と似たようなことができるからです。しかし、例えば、このクエリ

SHOW COLUMNS FROM {table} 

で、あなたの配列$fieldsに一致する入力を持つPOSTリクエストを想像してみてください。誰かが行う必要があるのは、値の代わりにキーにSQL Attackの部分を含むリクエストを送信することです。あなたは保護されません。このような何か（私はそれを言うべき感じ、バックアップせずにDBをドロップしません。）

$_POST['1=1; DROP DATABASE --'] = true; 

あなたのクエリを作成しますが

INSERT INTO table SET 1=1; DROP DATABASE -- = ?, {field} = ? .. rest of query 

--はSQLでコメントを開始しています何もないので、--のDBに関係しているので、エラーが発生するのを防ぐことができます。ちょうどあなたのSQLにポストキーをダンプすることに注意してください。1=1がうまくいくかはわかりませんが、 。これは単なる例示である。その場合は、私が何であるかを$this->queryか->error()知りませんが、私が想像するよう$this->queryが$thisまたは一部を返さなければなら必ずエラーリターンtrueないがあるかどうか私には

このライン

if($this->query($sql, $fields)->error()){ 
     return true; 
    }else{ 
     return false; 
    } 

は言いますエラーを含むオブジェクト。それはちょうど混乱の言葉をその方法です。