如何让PHP的代码更安全

吾爱主题阅读：153 2021-11-16 16:12:00 评论：0

概述

攻击者通过构造恶意SQL命令发送到数据库，如果程序未对用户输入的 SQL命令执行判断过滤，那么生成的SQL语句可能会绕过安全性检查，插入其他用于修改后端数据库的语句，并可能执行系统命令，从而对系统造成危害

例如删除 id 为 1 的帖子，sql 如下：

1 2 3 $post_id = $_POST [ 'post_id' ]; $sql = "DELETE FROM posts WHERE user_id = 1 AND id = $post_id" ; \DB::statement( $sql );

如果有人在提交 post_id 时输入 1 OR 1 ，你的语句会组合成这样：

1	`$sql` `=` `"DELETE FROM posts WHERE user_id = 1 AND id = 1 OR 1"` `;`

一般比较常出现在原生的 SQL 操作，框架一般会解决这方面的问题。通常采用参数控制或过滤特殊字符避免上述的问题。

越权漏洞

1. 水平越权

水平越权就是同等角色下的用户，不但能够访问和操作自己私有的数据，还能访问其他人私有的数据，其根本是基于数据的访问权限。

删除用户收款方式的场景如下：

graph LR

用户登录-->token

token-->获取收款方式列表

获取收款方式列表--token-->通过id删除

通过收款方式 {id} 执行 delete 请求的路由为：localhost/api/payments/{id}

假如用户A的收款方式有{1,2,3} 用户B的收款方式有{4,5}

如果没有做数据控制，A 登录后携带 A 的 token 执行删除的接口localhost/api/payments/4，则会删除 B 的，所以需要对destory方法做数据控制

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 # 1 删除前鉴权处理 public function destory( $id ){ $payment = Payment::find( $id ); if ( $payment ->user_id != $this ->currentUser->id) { return ... } $payment -> delete (); } # 2 参入id查询删除 public function destory( $id ){ Payment::whereUserId( $this ->currentUser->id)->whereId( $id )-> delete (); } # 3 模型关联查询 class User extends Model{ public function payments() { return $this ->hasMany( 'App\Payment' ); } } class PaymentController extends Controller{ public function destory( $id ) { $this ->currentUser->payments()->whereId( $id )-> delete (); } }

推荐使用第三种方式做数据控制，不然面向对象白学了。获取收款方式的列表同样需要数据权限控制，用户和收款方式存在一对多的关联关系，模型关联后，获取用户收款方式列表可以写成

1 2 3 4 5 6 7 8 9 10 11 12 class PaymentController extends Controller{ public function index( $id ) { #带条件的查询 $payments = $this ->currentUser->payments()->where( function ( $query ){ ... })->get(); #不带条件的查询 $payments = $this ->currentUser->payments; } }

2. 垂直越权

低权限的角色通过一些途径，获得高权限的能力，就发生了越权访问。如普通用户 guest 修改 admin 用户的密码；guest 可直接进入后台取得域名管理、用户管理等所有权限

解决这个问题，需要把权限职责以最小颗粒细分，基于 RBAC 设计权限管理系统。分为以下关联模型

graph LR

用户--多对多-->角色

用户--多对多-->权限

角色--多对多-->权限

每次执行请求时，在前置中间件判断这个用户是否永远该执行请求的权限，无权限则驳回。

3. 上下文越权

攻击者能够利用应用程序状态机中的漏洞获得关键资源的访问权限，这就存在上下文相关的越权。上下文相关的越权漏洞一般属于业务逻辑漏洞。如在找回密码过程中，攻击者使用自己的账户信息通过验证，将他人的密码进行了修改。

graph LR

1.邮箱验证-->2.找回密码

在步骤1之后，执行找回密码，路由为。如果此时没有校验当前找回密码的账户是否为进行邮箱校验后的账户，由可能产生越权漏洞.

路由 : 【PUT 】localhost/api/users/find-password，接收参数 email,new_password.

错误：校验和修改分成 2 步localhost/api/email/check->localhost/api/users/password

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 class UserController extends Controller{ public function check( $data ) { if (checkEmail( $data [ 'email' ], $data [ 'code' ])) { return true; } ... } public function findPassword() { $user = User::whereEmail( $data [ 'email' ])->first(); $user ->password = bcrypt( $data [ 'new_password' ]); $user ->save(); } }

正确：在 findPassword 里面再次验证完成邮箱校验的账户是否为当前找回密码的账号

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 class UserController extends Controller{ public function check( $data ) { if (checkEmail( $data [ 'email' ], $data [ 'code' ])) { return true; } ... } public function findPassword( $data ) { if (checkEmail( $data [ 'email' ], $data [ 'code' ])) { $user = User::whereEmail( $data [ 'email' ])->first(); $user ->password = $data [ 'new_password' ]; $user ->save(); } ... } }

限制分页条目范围，防止恶意请求

如获文章列表的接口localhost/api/articles

1 2 3 4 5 6 7 8 9 10 public function index( $params ){ $pageId = $params [ 'pageid' ] ?? PAGE_ID; //页码 $pageSize = $params [ 'pagesize' ] ?? 15; //条码 $articles = Article::where( function ( $query ) use ( $params ) { ... })->take( $pageSize )->skip( $pageId * $pageSize )->orderby( 'id' , 'desc' )->get(); ... ... }

以上代码如果没有限制 pagesize 的范围，恶意请求者请求把 pagesize 输入 5000,10000 等甚至更大的数，会给数据库带来一定的压力，localhost/api/articles?pageid=0&pagesize=10000

1 2 3 4 5 //用框架自带的分页方法 public function index(){ $builder = Article::with( 'category:id,name' )->orderBy( 'id' , 'desc' )->paginate(8); return response()->json([ 'status' => true, 'count' => $builder ->total(), 'articles' => $builder ->items()]); }

JWT 的 Token 需要二次加密

许多拓展包加密出的token并不十分安全，用base64_decode可以解密获取加密主键、载荷等重要信息，所以通常需要对JWT的token进行二次加密

限制上传文件的类型

对于一个图片上传的接口，如果没有对上传文件的格式做限制，攻击者很有可能把 .php后缀的文件上传到public/images目录下，然后通过根目录执行这个文件。

需要设计安全的文件上传功能避免上述问题

1.文件上传的目录设置为不可执行

2.判断文件类型

3.使用随机数改写文件名和文件路径

4.单独设置文件服务器的域名

禁止或者避免写自动解压.zip 等压缩文件的代码

单纯地限制文件或压缩包大小并没有用，一个ZIP炸弹的.zip文件仅有 42 KB，但在解压后会占用 4718592 GB

避免登录密码被暴力破解

1.设定严格的速率限制，如登录次数限制，登录错误次数达 x 次时暂停登录 n 分钟

2.密码加上随机盐

1 2 3 4 5 6 public function reg(){ $user = new User; $salt = radom(6); $user ->password = bcrypt( $data [ 'password' ] . $salt ); ... }

做好异常处理，避免在生产环境中不正确的错误报告暴露敏感数据

如果你不小心，可能会在生产环境中因为不正确的错误报告泄露了敏感信息，例如：文件夹结构、数据库结构、连接信息与用户信息。

1.在.env 文件中关闭调试模式

APP_DEBUG=true

2.php 错误控制 error_reporting、display_errors

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 <?php // 关闭错误报告 error_reporting (0); // 报告 runtime 错误 error_reporting (E_ERROR | E_WARNING | E_PARSE); // 报告所有错误 error_reporting (E_ALL); // 等同 error_reporting(E_ALL); ini_set ( "error_reporting" , E_ALL); // 报告 E_NOTICE 之外的所有错误 error_reporting (E_ALL & ~E_NOTICE); ?> display_errors = Off

php 弱语言的设计缺陷如：in_array

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 $array =[0,1,2, '3' ]; var_dump(in_array( 'abc' , $array )); //true var_dump(in_array( '1bc' , $array )); //true # 上面的情况返回的都是 true, 因为 'abc' 会转换为 0， '1bc' 转换为 1 $a = null; $b = false; echo $a == $b ; //true $c = "" ; $d = 0; echo $c == $d //true

在一些重要的地方需要使用===来作数据判断。

LFI (本地文件包含)

LFI （本地文件包含）是一个用户未经验证从磁盘读取文件的漏洞。

不验证过滤用户的输入将它要渲染的模板文件用 GET 请求加载。

1 2 3 4 5 6 7 8 <body> <?php $page = $_GET [ 'page' ]; if (! $page ) { $page = 'main.php' ; } include ( $page ); ?></body>

由于 Include 可以加载任何文件，不仅仅是 PHP，攻击者可以将系统上的任何文件作为包含目标传递。

index.php?page=../../etc/passwd

这将导致 /etc/passwd 文件被读取并展示在浏览器上。

要防御此类攻击，你必须仔细考虑允许用户输入的类型，并删除可能有害的字符，如输入字符中的 “.” “/” “\”。

XSS

XSS 又叫 CSS (Cross Site Script) ，跨站脚本攻击。它指的是恶意攻击者往 Web 页面里插入恶意 html 代码，当用户浏览该页之时，嵌入其中 Web 里面的 html 代码会被执行，从而达到恶意攻击用户的特殊目的。

1 2 3 4 5 <body> <?php $searchQuery = $_GET [ 'q' ]; /* some search magic here */ ?><h1>You searched for : <?php echo $searchQuery ; ?></h1></body>

因为我们把用户的内容直接打印出来，不经过任何过滤，非法用户可以拼接 URL： search.php?q=%3Cscript%3Ealert(1)%3B%3C%2Fscript%3E

PHP 渲染出来的内容如下，可以看到 Javascript 代码会被直接执行：

1 2 3 4 < body > < h1 >You searched for: < script >alert(1);</ script ></ h1 > < p >We found: Absolutely nothing because this is a demo</ p > </ body >