Commit 81715d00 authored by Piotr Caban's avatar Piotr Caban Committed by Alexandre Julliard

jscript: Add error handling to Array.reverse.

parent 8644a083
...@@ -473,17 +473,35 @@ static HRESULT Array_reverse(script_ctx_t *ctx, vdisp_t *vthis, WORD flags, DISP ...@@ -473,17 +473,35 @@ static HRESULT Array_reverse(script_ctx_t *ctx, vdisp_t *vthis, WORD flags, DISP
l = length-k-1; l = length-k-1;
hres1 = jsdisp_propget_idx(jsthis, k, &v1, ei, sp); hres1 = jsdisp_propget_idx(jsthis, k, &v1, ei, sp);
if(FAILED(hres1))
return hres1;
hres2 = jsdisp_propget_idx(jsthis, l, &v2, ei, sp); hres2 = jsdisp_propget_idx(jsthis, l, &v2, ei, sp);
if(FAILED(hres2)) {
VariantClear(&v1);
return hres2;
}
if(hres1 == DISP_E_UNKNOWNNAME) if(hres1 == DISP_E_UNKNOWNNAME)
jsdisp_delete_idx(jsthis, l); hres1 = jsdisp_delete_idx(jsthis, l);
else else
jsdisp_propput_idx(jsthis, l, &v1, ei, sp); hres1 = jsdisp_propput_idx(jsthis, l, &v1, ei, sp);
if(FAILED(hres1)) {
VariantClear(&v1);
VariantClear(&v2);
return hres1;
}
if(hres2 == DISP_E_UNKNOWNNAME) if(hres2 == DISP_E_UNKNOWNNAME)
jsdisp_delete_idx(jsthis, k); hres2 = jsdisp_delete_idx(jsthis, k);
else else
jsdisp_propput_idx(jsthis, k, &v2, ei, sp); hres2 = jsdisp_propput_idx(jsthis, k, &v2, ei, sp);
if(FAILED(hres2)) {
VariantClear(&v2);
return hres2;
}
} }
if(retv) { if(retv) {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment