sapplica / sentrifugo

Sentrifugo is a FREE and powerful Human Resource Management System (HRMS) that can be easily configured to meet your organizational needs.
http://www.sentrifugo.com/
GNU General Public License v3.0
484 stars 426 forks source link

sentrifugo opensource must not support sql injection attackers #357

Open manslogic opened 3 years ago

manslogic commented 3 years ago

As seen below screenshot. string concatenation used to build SQL queries and parametrized queries are not being used. This is very much support to hackers and not the right way of using PHP. Need to use parametrized queries where ever-dynamic queries are required and must not support hackers of any kind.

image

Example https://github.com/sapplica/sentrifugo/blob/master/application/modules/assets/models/AssetCategories.php