调用CloseHandle(句柄)处理NULL是否安全

Question

调用CloseHandle(句柄)处理NULL是否安全

删除NULL指针是安全的.

int* p = NULL;
delete p;         // ok, secure

Run Code Online (Sandbox Code Playgroud)

Handles有什么关系？

HANDLE h = NULL;
CloseHandle(h);   // allowed?

Run Code Online (Sandbox Code Playgroud)

我正在阅读MSDN但仍不确定.它说的是ERROR_INVALID_HANDLE,但是它是6L,而不是NULL.

我来自一个类的析构函数,它给了我一个C6387警告错误

if (m_hThread)
    WaitForSingleObject(m_hThread, INFINITE);
CloseHandle(m_hThread);    // warninig C6387
m_hThread = NULL;

Run Code Online (Sandbox Code Playgroud)

Answer 1

Mar*_*ica 8

不.您可能不会调用CloseHandleNULL句柄.查看参数的文档.它说:

hObject [in]打开对象的有效句柄.

NULL不是打开对象的句柄.

你得到C6387的事实正是因为你传递了一个"可能为空"的句柄CloseHandle.

您必须将代码编写为:

if (m_hThread) {
    WaitForSingleObject(m_hThread, INFINITE);
    CloseHandle(m_hThread);
}

Run Code Online (Sandbox Code Playgroud)

(在此之后没有必要将m_hThread设置为NULL - 它将在很短的时间内停止存在).

@TomTom您违反了API合同,该合同表明您必须传递有效的句柄.`NULL`不是有效的句柄.它甚至表示在调试模式下抛出异常.如果API合同明确,则没有理由根据假设进行编程.如果你想安全一些打字,只需创建一个单行:`inline BOOL CloseHandleSafe(HANDLE h){return h && h!= INVALID_HANDLE_VALUE？CloseHandle(h):TRUE;}`你可以在不先检查的情况下调用它. (5认同)
@TomTomt:这是安全的,因为你在dtor中调用它,而dtors没有被调用两次 (3认同)
有一篇文档经常被忽略:`hObject`参数上的[SAL注释](https://docs.microsoft.com/en-us/cpp/c-runtime-library/sal-annotations).此参数标记为`_In_`(与`_In_opt_`相对),因此不允许传递`nullptr`.它是SAL注释,导致编译器发出[C6387](https://docs.microsoft.com/en-us/visualstudio/code-quality/c6387)警告. (2认同)
@zett42，实际上NT 不会引发`NULL` 句柄的异常，无论是附加了调试器还是配置为始终这样做的全局系统标志。关闭 `NULL` 句柄只会导致调用失败并导致系统调用的开销。在实践中，不需要 SEH/VEH 处理程序是安全的。尽管显然不建议忽略警告，因为文档没有明确说明关闭“NULL”句柄与“关闭”伪句柄（例如“GetCurrentProcess()”）一样无操作（即相同作为`INVALID_HANDLE_VALUE`，在实践中）。 (2认同)
@TomTom 我不能做短于 3 行的：`inline BOOL CloseHandleSafe(HANDLE& h){ HANDLE tmp = h; h = NULL; 返回 tmp && tmp != INVALID_HANDLE_VALUE ? CloseHandle(tmp) : TRUE; }`。但是我不会使用这个函数，更好的是在析构函数中调用 `CloseHandle` 的 RAII 包装器。 (2认同)

归档时间：	8 年前
查看次数：	2363 次
最近记录：	8 年前