php - Blind Arithmetic Evaluation Differential : SQL Injection -


i getting sql & url injection vulnerabilities when scan website. code i'm using:

if(isset($_get["id"])) {     if(!is_int($_get["id"]) ==false)     {         //redirect person homepage     } else {         $sql = "select * workshop id=".trim($_get['id']);         $result = mysql_query($sql);          $row = mysql_fetch_assoc($result);         $id = $row['id'];            $prod_name = $row['prod_name'];          $description = $row['description'];          $image1 = $row['image1'];         $image2 = $row['image2'];         $image3 = $row['image3'];         $pdffilename = $row['pdffilename'];         $publish = $row['publish'];         $workshop_date = $row['workshop_date'];         $workshop_date_end = $row['workshop_date_end'];         $course_desc = $row['course_desc'];         $attend = $row['attend'];         $trainer_detail = $row['trainer_detail'];         $location = $row['location'];          $datevalue = $row['workshop_date'];         $year = date('y',strtotime($datevalue));         $month = date('f',strtotime($datevalue));         $day = date('d',strtotime($datevalue));          $datevalue1 = $row['workshop_date_end'];         $year1 = date('y',strtotime($datevalue1));         $month1 = date('f',strtotime($datevalue1));         $day1 = date('d',strtotime($datevalue1));     } }

how fix it?

the sql injection problem in row:

$sql = "select * workshop id=".trim($_get['id']);

you're applying value directly query without escaping it.

do instead:

$id = mysql_real_escape_string(trim($_get['id'])); $sql = "select * workshop id=$id";

remember you're using deprecated mysql_* functions, mysqli_* should used instead. consider updating code.


Comments

Post a Comment

Popular posts from this blog

get url and add instance to a model with prefilled foreign key :django admin -

Image
i have page in each contest shown in row. each row contains contest id , link other model names have foreign key contest. news model: class news(models.model): title = models.charfield(max_length=50, null=true) description = models.textfield() time = models.timefield( default= datetime.datetime.now().time()) contest = models.foreignkey(contest) i want use admin adding , changing these models. want when user clicks on 'news' has foreign key contest id=1, shows list of news value foreign key. use {% instance in instances %} <tr class="{% cycle 'row1' 'row2' %}"> <td>{{ instance.title }}</td> <td>{{ instance.starttime }}</td> <td>{{ instance.timespan }}</td> <td>{{ instance.date }}</td> <td><a href="/adm
Read more

css - Make div keyboard-scrollable in jQuery Mobile? -

from can tell, although jquery-mobile-powered pages can contain divs overflow set scroll or auto, , these divs can scrolled one-screen bar or mouse wheel, cannot scrolled using arrow keys, page-up/page-down, or home/end. instead, official "page" div (with data-role="page") absorbs input. perhaps other divs can't acquire focus, i'm not sure. is there way around this? edit: jsfiddle of simple example: https://jsfiddle.net/qogz0shx/ <script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.4/jquery.min.js"></script> <script src="https://ajax.googleapis.com/ajax/libs/jquerymobile/1.4.5/jquery.mobile.min.js"> </script> <link rel="stylesheet" href="https://code.jquery.com/mobile/1.4.5/jquery.mobile-1.4.5.css"/> <style> #outer { overflow:scroll; height: 50vh; width: 50vw; } #inner { height: 500vh; width: 500vw; } &l
Read more

ruby on rails - Seeing duplicate requests handled with Unicorn -

recently upgraded our ec2 instances c3.large in order boost more unicorn workers handle increased traffic our site. (6xworkers per machine - 2 ec2 instances). when did this, started seeing duplicate records being created! looks somehow, maybe 2 unicorn workers attempting process same request? in nginx logs see 1 [post] request particular rails controller - yet 2 records being created in database. i'm hitting kind of race condition issue - have no idea how debug this. doesn't happen time, when it's frustrating well. noticed 2 database records within 5 seconds of each other. (and yes disable button when user clicks submit button). unicorn 4.8.2 ruby 1.9.3 rails 3.2.14 any great! thanks! although may not fix problem, upgrade ruby 2.0.0. there blog mentioning idea upgrade ruby 2.0.0 if using unicorn. for example: https://www.digitalocean.com/community/articles/how-to-optimize-unicorn-workers-in-a-ruby-on-rails-app specifically: if using ruby 1.9,
Read more