[PATCH] advapi32: Eliminate dead stores (Clang).

Alexandre Julliard julliard at winehq.org
Fri Sep 14 04:39:41 CDT 2012


Charles Davis <cdavis5x at gmail.com> writes:

> @@ -615,9 +613,9 @@ static DWORD mac_write_credential(const CREDENTIALW *credential, BOOL preserve_b
>          attr_list.count = 0;
>          attr_list.attr = NULL;
>      }
> -    status = SecKeychainItemModifyAttributesAndData(keychain_item, &attr_list,
> -                                                    preserve_blob ? 0 : strlen(password),
> -                                                    preserve_blob ? NULL : password);
> +    SecKeychainItemModifyAttributesAndData(keychain_item, &attr_list,
> +                                           preserve_blob ? 0 : strlen(password),
> +                                           preserve_blob ? NULL : password);

I expect you'd want to handle that error instead.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list