Skip to content

Commit 7e58ec2

Browse files
committed
usersmodel - provide sql checks for inputs
* use form input library to check validity of the inputs so avoid the sql injection attacks not using specific dbms this permit to use any kind of database manager engine.
1 parent 5a5de9d commit 7e58ec2

File tree

1 file changed

+16
-0
lines changed

1 file changed

+16
-0
lines changed

webappweb/models/Usersmodel.php

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,14 @@ public function logindb($username, $password)
1212
{
1313
log_message('info', __METHOD__ .' begin ');
1414

15+
$validu = $this->form_validation->required($username);
16+
$validu = $this->form_validation->alpha_dash($username);
17+
$validu = $this->form_validation->max_length($username,40);
18+
$valids = $this->form_validation->required($password);
19+
$valids = $this->form_validation->alpha($password);
20+
21+
if($validu == FALSE AND $valids == FALSE) return FALSE;
22+
1523
$this->load->database();
1624
$query = $this->db->get_where('users', array('username'=>$username, 'userpass'=>$password));
1725
$array_result = $query->row_array();
@@ -24,6 +32,14 @@ public function loginimap($username, $password)
2432
{
2533
log_message('info', __METHOD__ .' begin ');
2634

35+
$validu = $this->form_validation->required($username);
36+
$validu = $this->form_validation->alpha_dash($username);
37+
$validu = $this->form_validation->max_length($username,40);
38+
$valids = $this->form_validation->required($password);
39+
$valids = $this->form_validation->alpha($password);
40+
41+
if($validu == FALSE AND $valids == FALSE) return FALSE;
42+
2743
$config = array('plain'=> TRUE, 'username' => $username, 'password' => $password);
2844
$this->load->library('Imap', $config);
2945
$valid = $this->imap->connect($config);

0 commit comments

Comments
 (0)