diff options
| author | Jan Beulich <jbeulich@suse.com> | 2013-03-27 08:46:28 +0100 |
|---|---|---|
| committer | Jan Beulich <jbeulich@suse.com> | 2013-03-27 08:46:28 +0100 |
| commit | 78380c34dfeb27da3d0222bcb7232c5d8e2f5b30 (patch) | |
| tree | e3c4e36b696255e6300979b43521f28264effe63 /xen/arch/x86/efi | |
| parent | 25250ed7c8094a905c5f03b8ae25c8694d3fa9b3 (diff) | |
| download | xen-78380c34dfeb27da3d0222bcb7232c5d8e2f5b30.tar.gz xen-78380c34dfeb27da3d0222bcb7232c5d8e2f5b30.tar.bz2 xen-78380c34dfeb27da3d0222bcb7232c5d8e2f5b30.zip | |
x86/EFI: permit setting variable with non-zero attributes
This must have been a copy-and-paste mistake - get_variable uses
op->misc as output only, and wants to make sure it's zero for future
extensibility. For set_variable, this is an input though, and hence
the check is wrong.
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Acked-by: Keir Fraser <keir@xen.org>
Diffstat (limited to 'xen/arch/x86/efi')
| -rw-r--r-- | xen/arch/x86/efi/runtime.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/xen/arch/x86/efi/runtime.c b/xen/arch/x86/efi/runtime.c index 73aa15eb9a..aafdfebdd8 100644 --- a/xen/arch/x86/efi/runtime.c +++ b/xen/arch/x86/efi/runtime.c @@ -398,9 +398,6 @@ int efi_runtime_call(struct xenpf_efi_runtime_call *op) long len; unsigned char *data; - if ( op->misc ) - return -EINVAL; - len = gwstrlen(guest_handle_cast(op->u.set_variable.name, CHAR16)); if ( len < 0 ) return len; |