881 Star 3.6K Fork 1.5K

Discuz / DiscuzX

 / 详情

php 7.4 提示 connect.php 10 有未定义的错误,虽然不知名,但是会有潜在危险

已完成
创建于  
2020-09-13 16:35

PHP Notice: Undefined index: mod in connect.php on line 10

大量这样的错误

评论 (6)

创建了任务
关联仓库设置为Discuz!/DiscuzX
展开全部操作日志

看到如下解决方案,不知道是否合适
因为Discuz! connect.php中使用了可能不存在的超全局变量$_GET的索引值。
打开connect.php文件,搜索

if($_GET['mod'] == 'register')

修改为

if(filter_has_var(INPUT_GET, 'mod') && $_GET['mod'] == 'register')

其实有个更简单的改法

if(isset($_GET['mod']) && $_GET['mod'] == 'register')

现在我这边优先处理的是在PHP 8上报Fatal Error的部分。
看看有没有人来提个PR修一修吧。

其实有个更简单的改法

if(isset($_GET['mod']) && $_GET['mod'] == 'register')

现在我这边优先处理的是在PHP 8上报Fatal Error的部分。
看看有没有人来提个PR修一修吧。

@老周部落 这个地方的特殊性在于connect.php是个入口文件,且一开始就出现了这个问题
error_reporting的设置在此处完全无法起作用,屏蔽不掉相关的错误提示。

@老周部落 这个地方的特殊性在于connect.php是个入口文件,且一开始就出现了这个问题
error_reporting的设置在此处完全无法起作用,屏蔽不掉相关的错误提示。

@popcorner 那看来还真是要改一下

那看来还真是要改一下

@老周部落 好像已经有PR了

@老周部落 好像已经有PR了

@popcorner 开完PR才看到...... 我关闭了。

湖中沉 通过Discuz/DiscuzX Pull Request !569任务状态待办的 修改为已完成

登录 后才可以发表评论

状态
负责人
里程碑
Pull Requests
关联的 Pull Requests 被合并后可能会关闭此 issue
分支
开始日期   -   截止日期
-
置顶选项
优先级
参与者(3)
1773794 laozhoubuluo 1594507411
PHP
1
https://gitee.com/Discuz/DiscuzX.git
git@gitee.com:Discuz/DiscuzX.git
Discuz
DiscuzX
DiscuzX

搜索帮助