]> git.sur5r.net Git - bacula/bacula/commitdiff
bacula-web: Replaced all $_POST by safe value in jobs page
authorDavide Franco <bacula-dev@dflc.ch>
Wed, 20 Jul 2011 16:43:39 +0000 (18:43 +0200)
committerKern Sibbald <kern@sibbald.com>
Sat, 20 Apr 2013 12:49:25 +0000 (14:49 +0200)
gui/bacula-web/jobs.php

index 12b6bd9b8fc06e92215af32fa6d045e99d9939ba..30cc2e915ff775f9c0f1b4e65b50609a00d57a4a 100644 (file)
   $query .= "FROM Job ";
   $query .= "LEFT JOIN Pool ON Job.PoolId=Pool.PoolId ";
   $query .= "LEFT JOIN Status ON Job.JobStatus = Status.JobStatus ";
-  
-  // Filter by status
-  if( isset( $_POST['status'] ) ) {
-       switch( $_POST['status'] ) 
+  $posts = CHttp::getRequestVars( $_POST );
+   
+  if( $posts != false ) {
+       switch( $posts['status'] ) 
        {
                case STATUS_RUNNING:
                        $query .= "WHERE Job.JobStatus = 'R' ";
@@ -67,7 +68,7 @@
                        $query .= "WHERE Job.JobStatus = 'A' ";
                break;
        }
-    $dbSql->tpl->assign('job_status_filter', $_POST['status'] );
+       $dbSql->tpl->assign('job_status_filter', $posts['status'] );
   }
   
   // order by
@@ -77,9 +78,9 @@
   $jobs_per_page = array( 25 => '25', 50 => '50', 75 => '75', 100 => '100', 150 => '150' );
     
   // Determine how many jobs to display
-  if( isset($_POST['jobs_per_page']) ) {
-       $query .= "LIMIT " . $_POST['jobs_per_page'];
-    $dbSql->tpl->assign( 'jobs_per_page_selected', $_POST['jobs_per_page'] );
+  if( isset($posts['jobs_per_page']) ) {
+       $query .= "LIMIT " . $posts['jobs_per_page'];
+    $dbSql->tpl->assign( 'jobs_per_page_selected', $posts['jobs_per_page'] );
   }else
        $query .= "LIMIT 25 ";
   $dbSql->tpl->assign( 'jobs_per_page', $jobs_per_page );
   $dbSql->tpl->assign( 'last_jobs', $last_jobs );
   
   // Count jobs
-  if( isset( $_POST['status'] ) )
-       $total_jobs = $dbSql->countJobs( FIRST_DAY, NOW, $_POST['status'] );
+  if( isset( $posts['status'] ) )
+       $total_jobs = $dbSql->countJobs( FIRST_DAY, NOW, $posts['status'] );
   else
        $total_jobs = $dbSql->countJobs( FIRST_DAY, NOW );