PHP 扩展 - 带有 PHP 发布编译的 Segfault

Question

我已经写了一个 PHP 扩展，但我现在真的很奇怪的段错误。

如果我通过php test.php或php < test.php运行测试脚本是可以的，但是如果我在交互模式下（php -a）输入完全相同的命令，就会出现段错误。

46              char *_class_name = (char *)emalloc(_class_name_len);
(gdb) s

Program received signal SIGSEGV, Segmentation fault.
0x000055555578f664 in _emalloc ()

我首先认为从 Launchpad 复制 PHP 二进制文件可能存在一些问题，我自己编译了一个。配置命令是'./configure' '--prefix=/opt/php7-dbg' '--with-gd' '--with-mysqli' '--with-readline' '--with-curl'。 （优化器参数是-O2。）

我又遇到了完全相同的问题，但这次我可以更进一步。

46              char *_class_name = (char *)emalloc(_class_name_len);
(gdb) s
_emalloc (size=4) at /home/frederick/Programming/C/php-7.0.6/Zend/zend_alloc.c:2439
2439    {
(gdb) n
2442            if (UNEXPECTED(AG(mm_heap)->use_custom_heap)) {
(gdb) n
2450            return zend_mm_alloc_heap(AG(mm_heap), size ZEND_FILE_LINE_RELAY_CC ZEND_FILE_LINE_ORIG_RELAY_CC);
(gdb) s
zend_mm_alloc_heap (size=4, heap=0x7fffef000040) at /home/frederick/Programming/C/php-7.0.6/Zend/zend_alloc.c:1365
1365            if (size <= ZEND_MM_MAX_SMALL_SIZE) {
(gdb) n
1366                    ptr = zend_mm_alloc_small(heap, size, ZEND_MM_SMALL_SIZE_TO_BIN(size) ZEND_FILE_LINE_RELAY_CC ZEND_FILE_LINE_ORIG_RELAY_CC);
(gdb) s
zend_mm_small_size_to_bin (size=4) at /home/frederick/Programming/C/php-7.0.6/Zend/zend_alloc.c:1211
1211            if (size <= 64) {
(gdb) n
1213                    return (size - !!size) >> 3;
(gdb) s
zend_mm_alloc_heap (size=<optimised out>, heap=0x7fffef000040) at /home/frederick/Programming/C/php-7.0.6/Zend/zend_alloc.c:1366
1366                    ptr = zend_mm_alloc_small(heap, size, ZEND_MM_SMALL_SIZE_TO_BIN(size) ZEND_FILE_LINE_RELAY_CC ZEND_FILE_LINE_ORIG_RELAY_CC);
(gdb) s
zend_mm_alloc_small (bin_num=<optimised out>, size=<optimised out>, heap=0x7fffef000040) at /home/frederick/Programming/C/php-7.0.6/Zend/zend_alloc.c:1286
1286                    size_t size = heap->size + bin_data_size[bin_num];
(gdb) n
1287                    size_t peak = MAX(heap->peak, size);
(gdb) n
1288                    heap->size = size;
(gdb) n
1289                    heap->peak = peak;
(gdb) n
1293            if (EXPECTED(heap->free_slot[bin_num] != NULL)) {
(gdb) n
1295                    heap->free_slot[bin_num] = p->next_free_slot;
(gdb) p bin_num
$1 = <optimised out>
(gdb) n

Program received signal SIGSEGV, Segmentation fault.
zend_mm_alloc_small (bin_num=<optimised out>, size=<optimised out>, heap=0x7fffef000040) at /home/frederick/Programming/C/php-7.0.6/Zend/zend_alloc.c:1295
1295                    heap->free_slot[bin_num] = p->next_free_slot;

我的代码中对应的函数是：

static std::string bnode_object_get_class_name(zval *object) {
    char *ini_ns_key = estrdup("bencode.namespace");
    zend_bool ini_ns = zend_ini_long(ini_ns_key, strlen(ini_ns_key), 0);
    efree(ini_ns_key);
    size_t _class_name_len = ZSTR_LEN(Z_OBJ_P(object)->ce->name);

    // segfault line!
    char *_class_name = (char *)emalloc(_class_name_len);

    strcpy(_class_name, ZSTR_VAL(Z_OBJ_P(object)->ce->name));
    std::string class_name(_class_name);
    efree(_class_name);
    if (ini_ns) {
        return class_name.substr(8);
    } else {
        return class_name;
    }
}

然后我知道哪条线路产生了段错误，我想解决这个问题。于是我立即编译了一个 PHP 的调试版本（配置命令：'./configure' '--prefix=/opt/php7-dbg' '--with-gd' '--with-mysqli' '--with-readline' '--with-curl' '--enable-debug'，优化器：-O0）。

但是在基于相同代码的调试版本中，段错误就消失了！

我不是一个经验丰富的 C/C++ 开发人员，这是我第一次遇到这样的问题。请帮忙，非常感谢。

更新

看来问题是由一个愚蠢的错误引起的。导致段错误的行应该是

char *_class_name = (char *)emalloc(_class_name_len + 1);

因为 C 字符串应该以 '\0' 结尾

但为什么调试版本没问题？

Answer 1

我的猜测是你的程序在strcpy 崩溃，emalloc 行只是崩溃前的最后一行。你真的有一个错误，需要在 emalloc 中添加 +1，因为字符串末尾有 0 个字节。

但是你把它弄得太复杂了。由于您使用的是 C++，因此您可以用一行替换从 size_t 到 efree 的所有行（包括它们）：

std::string class_name(ZSTR_VAL(Z_OBJ_P(object)->ce->name), ZSTR_LEN(Z_OBJ_P(object)->ce->name));

这也是防崩溃的。

另外，你把函数的开头复杂化了：

char *ini_ns_key = estrdup("bencode.namespace"); zend_bool ini_ns = zend_ini_long(ini_ns_key, strlen(ini_ns_key), 0); efree(ini_ns_key);

应该这样写：

zend_long ini_ns_long = zend_ini_long("bencode.namespace", strlen("bencode.namespace"), 0); bool ini_ns=(ini_ns_long!=0);