C# выявляет проблему CredWrite в Windows XP Pro

В настоящее время у меня есть программа, написанная на C#, вызывающая CredRead/CredWrite из Advapi32.dll, чтобы пользователям, не присоединенным к домену, было легко сохранять учетные данные сервера терминалов / сервера WebDAV. ( http://msdn.microsoft.com/en-us/library/aa374788%28v=vs.85%29.aspx для информации о структуре учетных данных)

Это все работает нормально, но на днях я столкнулся с проблемой, которая сохраняется во всех операционных системах XP; Я не могу сохранить учетные данные сервера терминалов. Собственный метод CredWrite возвращает код ошибки 87 (ERROR_INVALID_PARAMETER).

Попробовав множество различных комбинаций сохраняемости учетных данных и типов, я понял, что проблема заключается в самом TargetName. Чтобы сохранить терминальные серверы в связке ключей Windows, TargetName - TERMSRV/server.domain.com. В Windows Vista или выше это нормально работает с моим кодом, а в Windows XP - нет.

Как ни странно, когда вы запускаете приложение удаленного рабочего стола в Windows XP и сохраняете учетные данные, оно делает это без суеты. Когда я создал метод для перечисления сохраненных учетных данных после использования RDP (очевидно, что пароли паролей не возвращались), я мог видеть, что цель была идентична цели, которую моя программа пыталась написать.

Чтобы убедиться, что проблема была в TargetName, я удалил "/", и он работал нормально.

Ниже приведена соответствующая часть моего кода:

var writeInt = NativeCredMan.WriteCred("TERMSRV/host.server.com", samAccountName, password, CRED_TYPE.DOMAIN_PASSWORD, CRED_PERSIST.LOCAL_MACHINE);

......

public enum CRED_TYPE : uint
{
    GENERIC = 1,
    DOMAIN_PASSWORD = 2,
    DOMAIN_CERTIFICATE = 3,
    DOMAIN_VISIBLE_PASSWORD = 4,
    GENERIC_CERTIFICATE = 5,
    DOMAIN_EXTENDED = 6,
    MAXIMUM = 7,      // Maximum supported cred type
    MAXIMUM_EX = (MAXIMUM + 1000),  // Allow new applications to run on old OSes
}

public enum CRED_PERSIST : uint
{
    SESSION = 1,
    LOCAL_MACHINE = 2,
    ENTERPRISE = 3,
}

......

    [DllImport("Advapi32.dll", EntryPoint = "CredReadW", CharSet = CharSet.Unicode, SetLastError = true)]
    static extern bool CredRead(string target, CRED_TYPE type, int reservedFlag, out IntPtr CredentialPtr);

    [DllImport("Advapi32.dll", EntryPoint = "CredWriteW", CharSet = CharSet.Unicode, SetLastError = true)]
    static extern bool CredWrite([In] ref NativeCredential userCredential, [In] UInt32 flags);

    [DllImport("Advapi32.dll", EntryPoint = "CredFree", SetLastError = true)]
    static extern bool CredFree([In] IntPtr cred);

    [DllImport("Advapi32.dll", EntryPoint = "CredDeleteW", CharSet = CharSet.Unicode)]
    static extern bool CredDelete(string target, CRED_TYPE type, int flags);

    //[DllImport("advapi32", SetLastError = true, CharSet = CharSet.Unicode)]
    //static extern bool CredEnumerateold(string filter, int flag, out int count, out IntPtr pCredentials);

    [DllImport("advapi32.dll", SetLastError = true, CharSet = CharSet.Unicode)]
    public static extern bool CredEnumerate(string filter, uint flag, out uint count, out IntPtr pCredentials);

    [StructLayout(LayoutKind.Sequential, CharSet = CharSet.Unicode)]
    private struct NativeCredential
    {
        public UInt32 Flags;
        public CRED_TYPE Type;
        public IntPtr TargetName;
        public IntPtr Comment;
        public System.Runtime.InteropServices.ComTypes.FILETIME LastWritten;
        public UInt32 CredentialBlobSize;
        public IntPtr CredentialBlob;
        public UInt32 Persist;
        public UInt32 AttributeCount;
        public IntPtr Attributes;
        public IntPtr TargetAlias;
        public IntPtr UserName;

        internal static NativeCredential GetNativeCredential(Credential cred)
        {
            var ncred = new NativeCredential
                            {
                                AttributeCount = 0,
                                Attributes = IntPtr.Zero,
                                Comment = IntPtr.Zero,
                                TargetAlias = IntPtr.Zero,
                                Type = CRED_TYPE.DOMAIN_PASSWORD,
                                Persist = (UInt32) cred.Persist,
                                CredentialBlobSize = (UInt32) cred.CredentialBlobSize,
                                TargetName = Marshal.StringToCoTaskMemUni(cred.TargetName),
                                CredentialBlob = Marshal.StringToCoTaskMemUni(cred.CredentialBlob),
                                UserName = Marshal.StringToCoTaskMemUni(cred.UserName)
                            };
            return ncred;
        }
    }

    [StructLayout(LayoutKind.Sequential, CharSet = CharSet.Unicode)]
    public struct Credential
    {
        public UInt32 Flags;
        public CRED_TYPE Type;
        public string TargetName;
        public string Comment;
        public System.Runtime.InteropServices.ComTypes.FILETIME LastWritten;
        public UInt32 CredentialBlobSize;
        public string CredentialBlob;
        public CRED_PERSIST Persist;
        public UInt32 AttributeCount;
        public IntPtr Attributes;
        public string TargetAlias;
        public string UserName;
    }

......

        public static int WriteCred(string key, string userName, string secret, CRED_TYPE type, CRED_PERSIST credPersist)
    {

        var byteArray = Encoding.Unicode.GetBytes(secret);
        if (byteArray.Length > 512)
            throw new ArgumentOutOfRangeException("The secret message has exceeded 512 bytes.");

        var cred = new Credential
                       {
                           TargetName = key,
                           CredentialBlob = secret,
                           CredentialBlobSize = (UInt32) Encoding.Unicode.GetBytes(secret).Length,
                           AttributeCount = 0,
                           Attributes = IntPtr.Zero,
                           UserName = userName,
                           Comment = null,
                           TargetAlias = null,
                           Type = type,
                           Persist = credPersist
                       };
        var ncred = NativeCredential.GetNativeCredential(cred);

        var written = CredWrite(ref ncred, 0);
        var lastError = Marshal.GetLastWin32Error();
        if (written)
        {
            return 0;
        }
        var message = "";
        if (lastError == 1312)
        {
            message = (string.Format("Failed to save " + key + " with error code {0}.", lastError) + "  This error typically occurrs on home editions of Windows XP and Vista.  Verify the version of Windows is Pro/Business or higher.");
        }
        else
        {
            message = string.Format("Failed to save " + key + " with error code {0}.", lastError);
        }
        MessageBox.Show(message);
        return 1;
    }

Может ли кто-нибудь указать в правильном направлении, как я могу сохранить эти типы учетных данных для сохранения?

Спасибо

1 ответ

Не берите в голову. Я нашел проблему. Изучив мой вопрос, я заметил, что управляемый счетчик учетных данных здесь:

internal static NativeCredential GetNativeCredential(Credential cred)
    {
        var ncred = new NativeCredential
                        {
                            AttributeCount = 0,
                            Attributes = IntPtr.Zero,
                            Comment = IntPtr.Zero,
                            TargetAlias = IntPtr.Zero,
                            Type = CRED_TYPE.DOMAIN_PASSWORD,
                            Persist = (UInt32) cred.Persist,
                            CredentialBlobSize = (UInt32) cred.CredentialBlobSize,
                            TargetName = Marshal.StringToCoTaskMemUni(cred.TargetName),
                            CredentialBlob = Marshal.StringToCoTaskMemUni(cred.CredentialBlob),
                            UserName = Marshal.StringToCoTaskMemUni(cred.UserName)
                        };
        return ncred;
    }

для TYPE было предопределено значение CRED.TYPE.DOMAIN_PASSWORD вместо простого удаления типа.

Изменение на:

internal static NativeCredential GetNativeCredential(Credential cred)
    {
        var ncred = new NativeCredential
                        {
                            AttributeCount = 0,
                            Attributes = IntPtr.Zero,
                            Comment = IntPtr.Zero,
                            TargetAlias = IntPtr.Zero,
                            Type = cred.type,
                            Persist = (UInt32) cred.Persist,
                            CredentialBlobSize = (UInt32) cred.CredentialBlobSize,
                            TargetName = Marshal.StringToCoTaskMemUni(cred.TargetName),
                            CredentialBlob = Marshal.StringToCoTaskMemUni(cred.CredentialBlob),
                            UserName = Marshal.StringToCoTaskMemUni(cred.UserName)
                        };
        return ncred;
    }

и отправка учетных данных в качестве ОБЩАЯ решала проблему. По-видимому, в Windows Vista/7 было разрешено использовать "/" для постоянного пароля DOMAIN_PASSWORD и GENERIC, но XP разрешает это только для GENERIC.

Другие вопросы по тегам