Security, adding intval in queries

skala
Julio Montoya 14 years ago
parent 7020474432
commit 22ce596a84
  1. 8
      main/announcements/announcements.inc.php

@ -1231,7 +1231,10 @@ class AnnouncementManager {
public static function send_email($id) {
$sent_to = self::sent_to("announcement", $id);
$userlist = $sent_to['users'];
$userlist = array_map('intval', $userlist);
$grouplist = $sent_to['groups'];
$grouplist = array_map('intval', $grouplist);
$course_id = api_get_course_int_id();
$course_code = api_get_course_id();
@ -1244,7 +1247,7 @@ class AnnouncementManager {
$tbl_announcement_attachment = Database::get_course_table(TABLE_ANNOUNCEMENT_ATTACHMENT);
// groepen omzetten in users
if ($grouplist) {
if (isset($grouplist) && !empty($grouplist)) {
$grouplist = "'".implode("', '",$grouplist)."'"; //protect individual elements with surrounding quotes
$sql = "SELECT user_id
FROM $tbl_groupUser gu
@ -1257,7 +1260,7 @@ class AnnouncementManager {
}
}
if (is_array($userlist)) {
if (isset($userlist) && !empty($userlist) && is_array($userlist)) {
$userlist = "'".implode("', '", array_unique($userlist) )."'";
// send to the created 'userlist'
@ -1282,7 +1285,6 @@ class AnnouncementManager {
$tbl_session_course_user.id_session = ".api_get_session_id();
}
}
$user_info = api_get_user_info();
if ($sqlmail != '') {

Loading…
Cancel
Save