I use codeigniter internally to develop our web solutions for somewhere in… net limited, a Norwegian company. Day before yesterday we suffered a terrible situation for an internal bug in code igniter which corrupted data inside some tables of our application database and then it tooks hours to find the origin of that bug, to fix it and to repair the corrupted data. Let me explain what happened.
Lets guess that we have one table named “users” with the following field
1. id
2. username
3. password
4. email
At some point, if you want to update the password field of this table, for a particular user, what will you write in your code?
$data = array("password"=>$new_password);
$this->db->where("id",$user_id);
$this->db->update("users", $data);
CodeIgniter’s ORM will create the query like the following
UPDATE users set password='{$new_password}' where user_id='{$user_id}';
Well, it’s ok and the quesry seems pretty fine. Now what should happen if you pass a valid user id to this code? Password of only that user will be updated. But what will happen when the passed $user_id is null?? Thats the most pathetic part that Codeigniter ORM plays. Instead of generating the following query,
UPDATE users set password='{$new_password}' where user_id=;
CodeIgniter’s ORM actually generates the following
UPDATE users set password='{$new_password}' where user_id;
You find the difference of the above two queries right? one contains “where user_id=’‘ ” and another contains just “where user_id” . Now if your backend database is MySQL and this query executes? You know what the hell will happen? It will replace all the user’s password with this new password instead of failing as MySQL count the “where user_id” part equals to false and returns all users. But If your Database is PostgreSQL, it fails, you are lucky.
So day before yesterday we suffered this problem against our commercial application which corrupts our user profile data. We immediate fixed the issue from our backup db (well, we lost 3 data) and then we started to find out what actually went wrong and found this vulnerable bug in CI.
So we suggest the CodeIgniter team to fix the issue immediately and change their ORM code so that it creates the query like the following if the value of passed argument is null. because it will fail to execute in all db. Otherwise the fellow user’s of code igniter, prepare for the dooms day.
UPDATE users set password='{$new_password}' where user_id=;
