On 2015/12/17 18:00, Ard Biesheuvel wrote:
Clang does not allow straight assignment of va_list types, so use the
correct VA_COPY() instead.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ard Biesheuvel <ard.biesheu...@linaro.org>
---
MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
First, the patch is good to me.
Reviewed-by: Star Zeng <star.z...@intel.com>
But Andrew ever has sent same code change and passed code review before.
http://article.gmane.org/gmane.comp.bios.edk2.devel/4160/
http://article.gmane.org/gmane.comp.bios.edk2.devel/3989/
As I know, Andrew has the permission to commit code, but he did not
commit that code yet.
You and Andrew need my help to commit the code with just Andrew's
Signed-off-by or with your both's Signed-off-by?
Thanks,
Star
diff --git a/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c
b/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c
index 621ce16da006..0c18d195215a 100644
--- a/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c
+++ b/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c
@@ -1716,7 +1716,7 @@ CheckRemainingSpaceForConsistencyInternal (
ASSERT_EFI_ERROR (Status);
TotalNeededSize = 0;
- Args = Marker;
+ VA_COPY(Args, Marker);
VariableEntry = VA_ARG (Args, VARIABLE_ENTRY_CONSISTENCY *);
while (VariableEntry != NULL) {
//
@@ -1745,7 +1745,7 @@ CheckRemainingSpaceForConsistencyInternal (
return FALSE;
}
- Args = Marker;
+ VA_COPY(Args, Marker);
VariableEntry = VA_ARG (Args, VARIABLE_ENTRY_CONSISTENCY *);
while (VariableEntry != NULL) {
//
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel