ext/date truncates error messages with null bytes
未关闭
还没有人认领这个 Issue。
Bug
Extension: date
Status: Verified
- 主要语言
- C
- 星标
- 40.4k
- 派生
- 8.2k
- 平均合并
- 2 天 13 小时
- 30 天内合并 PR
- 96
描述
date_throw_uninitialized_error() (C function):
<?php
$o = new class extends DateTime {
public function __construct() {}
};
var_dump(get_class($o));
$o->format("");
OnUpdate_date_timezone ini handler:
<?php
ini_set("date.timezone", "foo\0bar");
php_date_initialize() (C function):
<?php
new DateTimeImmutable("foo\0bar");
php_date_modify() (C function):
<?php
$now = new DateTime();
$now->modify("foo\0bar");
date_interval_initialize() (C function):
<?php
new DateInterval("foo\0bar");
php_date_interval_initialize_from_hash() (C function):
<?php
var_dump(unserialize("O:12:\"DateInterval\":2:{s:11:\"from_string\";b:1;s:11:\"date_string\";s:7:\"foo\0bar\";}"));
date_interval_create_from_date_string() (C function):
<?php
date_interval_create_from_date_string("foo\0bar");
DateInterval::createFromDateString():
<?php
DateInterval::createFromDateString("foo\0bar");
date_period_initialize() (C function):
<?php
DatePeriod::createFromISO8601String("foo\0bar");
date_period_initialize() (C function):
<?php
DatePeriod::createFromISO8601String("foo\0bar");
date_default_timezone_set():
<?php
date_default_timezone_set("foo\0bar");
date_period_unset_property() (C function):
<?php
$o = new class() extends DatePeriod {
public function __construct() {}
};
unset($o->start);
Tested with 3v4l
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
从 ext/date 的实现开始,定位到指定的 C 函数,并使用提供的 PHP 代码片段重现列出的 null 字节情况。跟踪每条错误消息的构建方式,然后为受影响的路径添加覆盖,并验证生成的消息保留相关输入;运行 date 扩展测试,以确认所有列出的入口点。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- c, php
- 领域
- backend
- Issue 类型
- 缺陷
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 活跃度
- 冷清
- 描述清晰度
- 基本清楚
- 新手友好度
- 55/100