网上看到的DisableWriteProtect和EnableWriteProtect这两个函数的实现有问题吗？

VOID DisableWriteProtect(PULONG pOldAttr)
{
    ULONG uAttr;
    _asm
    {
        push eax
            mov eax, cr0
            mov uAttr, eax
            and eax, 0FFFEFFFFh // CR0 16 BIT = 0
            mov cr0, eax
            pop eax
    };
    *pOldAttr = uAttr; //保存原有的 CRO 属性

}

VOID EnableWriteProtect(ULONG uOldAttr )
{
    _asm
    {
        push eax
            mov eax, uOldAttr //恢复原有 CR0 属性
            mov cr0, eax
            pop eax
    };

   }