[PATCH] mshtml: specify correct variable in TRACE (Coverity)

Marcus Meissner marcus at jet.franken.de
Thu May 9 03:22:06 CDT 2013


747429 Out-of-bounds read

Use prop_vt as apparently intended, i is over the end
of the array.

Ciao, Marcus
---
 dlls/mshtml/dispex.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/mshtml/dispex.c b/dlls/mshtml/dispex.c
index d70ae2d..54154aa 100644
--- a/dlls/mshtml/dispex.c
+++ b/dlls/mshtml/dispex.c
@@ -267,7 +267,7 @@ static void add_func_info(dispex_data_t *data, DWORD *size, tid_t tid, const FUN
 
         info->prop_vt = desc->elemdescFunc.tdesc.vt;
         if(info->prop_vt != VT_VOID && !is_arg_type_supported(info->prop_vt)) {
-            TRACE("%s: return type %d\n", debugstr_w(info->name), info->arg_types[i]);
+            TRACE("%s: return type %d\n", debugstr_w(info->name), info->prop_vt);
             return; /* Fallback to ITypeInfo::Invoke */
         }
 
-- 
1.7.10.4




More information about the wine-patches mailing list