php - Passing $_POST to Db Class Without any Processing -


i trying pass $_post global variable directly database through mysqli_real_escape_string without processing...

here code working fine.

  1. what possible problem arising due type of coding?
  2. what security risks?
  3. is shortcut or there still more...???
class db{ ... more ...      public function insert($args=array()){         if(!isset($args['table']) || !isset($args['values'])){             return false;         }         $table=$args['table'];         $values=$this->process_insert($args['values']);         $query="insert {$table} {$values}";         $result=$this->query2db($query); //method returns mysqli_query()         if($result){             return true;         }else{             return false;         }     }      protected function process_insert($data=array()){         $values="";         $fields="";         $glue=", ";         if(isset($data)){             foreach($data $key=> $value){                 $key=$this->escape($key);                 $value=$this->escape($value); //method returns mysqli_real_escape_string                 $fields .="{$key}".$glue;                 $values .="'{$value}'".$glue;             }             $fields=rtrim($fields, $glue);             $values=rtrim($values, $glue);             return "({$fields}) values ({$values})";         }else{             return false;         }     }  ... more ... }  if(!empty($_post)){      $query=array(     'table'=>'users',     'values'=> $_post //<=== problem here     );      $dbobj=new db;     $dbobj->insert($query);  }

update-

  • i can use interactions android...
  • auto generate form having values in array format
  • and validating using array

instead of using mysql_real_escape_string, suggest using prepared statements. say, never trust users input. data escaped still gets in database , not forget performance boost using prepared statements!

read post mysql_real_escape_string vs prepared statements


Comments

Post a Comment

Popular posts from this blog

php - How to add and update images or image url in Volusion using Volusion API -

actually have store importing products store product data insert , update image or image url not insert , update. this xml code name datapro.txt. <products_joined> <productcode>3710_012t</productcode> <vendor_partno>eah5450silent/di/1gd3(lp)</vendor_partno> <productname>test product ta</productname> <hideproduct>n</hideproduct> <stockstatus>20</stockstatus> <lastmodified>1/5/2016 10:25:00 am</lastmodified> <lastmodby>2</lastmodby> <productweight>0.9</productweight> <productprice>100</productprice> <productmanufacturer>asus tek</productmanufacturer> <vendor_price>32.69</vendor_price> <numproductssharingstock>0</numproductssharingstock> <categoryids>107</categoryids> <producturl>http://tebkq.mvlce.servertrust.com/productdetails.asp?productcode=3710_012t</producturl...
Read more

Laravel mail error `Swift_TransportException in StreamBuffer.php line 269: Connection could not be established with host smtp.gmail.com [ #0]` -

i developing website contact-us page guest users can send me email using gmail . anytime try send mail using contact-us form, error swift_transportexception in streambuffer.php line 269: connection not established host smtp.gmail.com [ #0] this mail.php return [ 'driver' => env('mail_driver', 'smtp'), 'host' => env('mail_host', 'smtp.gmail.com'), 'port' => 587, 'from' => ['address' => '', 'name' => ''], 'encryption' => env('mail_encryption', 'tls'), 'username' => env('mail_username'), 'password' => env(mail_password), 'sendmail' => '/usr/sbin/sendmail -bs', 'pretend' => env('mail_pretend', false), ]; this .env mail_driver=smtp mail_host=smtp.gmail.com mail_port=587 mail_username=afecjaped@gmail.com mail_password=********...
Read more

c# SetCompatibleTextRenderingDefault must be called before the first -

i tried search exception couldn't find solution on case i'am using code below invoke .net application : assembly assem = assembly.load(data); methodinfo method = assem.entrypoint; var o = activator.createinstance(method.declaringtype); method.invoke(o, null); the application invoked has form , in entrypoint of application : [stathread] static void main() { application.enablevisualstyles(); application.setcompatibletextrenderingdefault(false); //exception application.run(new form1()); } setcompatibletextrenderingdefault must called before first iwin32window object created in application. edit : assembly = assembly.load(data); methodinfo method = a.gettype().getmethod("start"); var o = activator.createinstance(method.declaringtype); method.invoke(o, null); you should create new method skips initialization...
Read more