编程规范，安全等相关问题

jiony

ErrorCode GetMenuFreshTime(OUT U16 *pu16MenuFreshTime )
{
        ErrorCode         ErrorCode = ErrorNoError;
        BoxInfo        *mpBoxInfo;

//        pu16MenuFreshTime = NULL ;//这里到底需要初始化不？
       
        ErrorCode = GetBoxInfo(&mpBoxInfo);       

        if (ErrorCode != DBSM_ErrorNoError)
        {
                return ErrorCode;
        }
        else
        {
                *pu16MenuFreshTime = mpBoxInfo ->mu16MenuFreshTime;
                return ErrorNoError;
        }
       
        FreeBoxInfo(mpBoxInfo);
        return ErrorNoError;

}

今天编写一个小的功能函数的时候出现了很多问题：
1.红色部分到底需要初始话不？这个函数是用来传出值的。
2.如果函数是二级指针就是**p，那么是否（必须）在函数内申请空间并初始化。
3.这个程序是否还有其他问题。执行到else的时候会不会不执行FreeBoxInfo(mpBoxInfo);
        return ErrorNoError;这些内容，直接跳出函数了，因为有个return ErrorNoError;

cjaizss

如果都
pu16MenuFreshTime = NULL ;
了，
那还玩个飞机

jiony

感谢楼上回复，可是没有说清楚

welkin

回答：
1.你的第一个return前没有对pu16MenuFreshTime赋值，所以是需要赋值的。但是你需要赋值的是*pu16MenuFreshTime ,而不是pu16MenuFreshTime。
2.如果传入参数是二级指针，如果你在函数外面已经分配空间了，应该就没有必要了吧。

jiony

楼上的有道理，感谢~

hyrish

回答问题3:
      执行到else部分后,根本不会执行

FreeBoxInfo(mpBoxInfo);

   程序存在内存泄漏,程序对错误处理的逻辑上还存在一些问题

[ 本帖最后由 hyrish 于 2007-11-9 13:32 编辑 ]