PHP扩展 - 使用PHP发布编译的Segfault

时间:2016-05-20 05:03:49

标签: php c++ c segmentation-fault php-extension

我已经编写了一个PHP扩展,现在我真的非常奇怪了。

如果我按php test.phpphp < 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命令为'./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)。

但是使用调试版本,基于相同的代码,segfault就会消失!

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

更新

似乎这个问题是由一个愚蠢的错误造成的。导致段错误的行应

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

因为C字符串应以'\ 0'

结束

但为什么调试版本没问题呢?

1 个答案:

答案 0 :(得分:0)

我的猜测是你的程序在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);