mshtml: Fix return type value for HTMLBodyElement_put_text.

Gerald Pfeifer gerald at pfeifer.com
Sat May 8 06:07:49 CDT 2010


The current code does not handle an error coming out of 
nsIDOMHTMLBodyElement_SetText and actually discards whatever nsres
is set to.

Jacek, is a patch like this okay, should we just remove nsres, or
is it more complicated than that?

Gerald
---
 dlls/mshtml/htmlbody.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/dlls/mshtml/htmlbody.c b/dlls/mshtml/htmlbody.c
index e4bb1d0..71b0b35 100644
--- a/dlls/mshtml/htmlbody.c
+++ b/dlls/mshtml/htmlbody.c
@@ -435,7 +435,7 @@ static HRESULT WINAPI HTMLBodyElement_put_text(IHTMLBodyElement *iface, VARIANT
     nsres = nsIDOMHTMLBodyElement_SetText(This->nsbody, &text);
     nsAString_Finish(&text);
 
-    return S_OK;
+    return nsres;
 }
 
 static HRESULT WINAPI HTMLBodyElement_get_text(IHTMLBodyElement *iface, VARIANT *p)
-- 
1.6.6.2



More information about the wine-patches mailing list