skyeye1.2.9源代码--疑似错误

aguang

在看skyeye1.2.9的源代码的时候，发现有个地方好像有错误，发上来给大家鉴定一下

在目录  skyeye-1.2.9_rc1\common\conf_parser  下有个文件  skyeye_options.c
从第99行开始的代码是：

    node->option_name = skyeye_strdup(option_name);
    if(node->option_name == NULL){
        skyeye_free(node);
        return Malloc_exp;
    }
    node->do_option = do_option_func;
    /* maybe we should use skyeye_mm to replace all the strdup */
    node->helper = skyeye_strdup(helper);
    if(node->option_name == NULL){
        skyeye_free(node->option_name);
        skyeye_free(node);
        return Malloc_exp;
    }


我认为红色部分应该改为：
if(node->helper == NULL){
作为对skyeye_strdup()函数返回值的确认。

不知道对不对，欢迎指正。

[ 本帖最后由 aguang 于 2009-10-19 19:26 编辑 ]

ksh

恩，笔误，已经改正