Better input checking on GET and POST vars

thomascube

2007-02-16 b3ce7915610a6d272cc38ecd2a8b61e04ee4aeae

 program/steps/mail/list.inc

@@ -22,10 +22,8 @@
$REMOTE_REQUEST = TRUE;
$OUTPUT_TYPE = 'js';

$sort = isset($_GET['_sort']) ? $_GET['_sort'] : false;

// is there a sort type for this request?
if ($sort)
if ($sort = get_input_value('_sort', RCUBE_INPUT_GET))
  {
  // yes, so set the sort vars
  list($sort_col, $sort_order) = explode('_', $sort);
@@ -40,22 +38,13 @@
  $sort_col   = isset($_SESSION['sort_col'])   ? $_SESSION['sort_col']   : $CONFIG['message_sort_col'];
  $sort_order = isset($_SESSION['sort_order']) ? $_SESSION['sort_order'] : $CONFIG['message_sort_order'];
  }
  

// we have a saved search request
if (!empty($_GET['_search']) && isset($_SESSION['search'][$_GET['_search']]))
  {
  $a_msgs = split(',', $_SESSION['search'][$_GET['_search']]);
  $a_headers = $IMAP->list_header_set($mbox, $a_msgs, NULL, $sort_col, $sort_order);
  $count = count($a_msgs);
  }
else
  {
  if ($count = $IMAP->messagecount())
    $a_headers = $IMAP->list_headers($mbox, NULL, $sort_col, $sort_order);
  }

$unseen = $IMAP->messagecount($mbox, 'UNSEEN', !empty($_GET['_refresh']) ? TRUE : FALSE);
// fetch message headers
if ($count = $IMAP->messagecount())
  $a_headers = $IMAP->list_headers($mbox_name, NULL, $sort_col, $sort_order);

$unseen = $IMAP->messagecount($mbox_name, 'UNSEEN', !empty($_GET['_refresh']) ? TRUE : FALSE);

// update message count display
$pages = ceil($count/$IMAP->page_size);
@@ -64,8 +53,8 @@
$commands .= sprintf("this.set_rowcount('%s');\n", rcmail_get_messagecount_text($count));

// update mailboxlist
$mbox = $IMAP->get_mailbox_name();
$commands .= sprintf("this.set_unread_count('%s', %d);\n", addslashes($mbox), $unseen);
$mbox_name = $IMAP->get_mailbox_name();
$commands .= sprintf("this.set_unread_count('%s', %d);\n", addslashes($mbox_name), $unseen);


// add message rows