Commit ac41380b authored by Jacek Caban's avatar Jacek Caban Committed by Alexandre Julliard

jscript: Fixed some Coverity issues.

parent df53b501
...@@ -325,7 +325,7 @@ static HRESULT array_join(script_ctx_t *ctx, jsdisp_t *array, DWORD length, cons ...@@ -325,7 +325,7 @@ static HRESULT array_join(script_ctx_t *ctx, jsdisp_t *array, DWORD length, cons
TRACE("= %s\n", debugstr_jsstr(ret)); TRACE("= %s\n", debugstr_jsstr(ret));
if(r) if(r)
*r = ret ? jsval_string(ret) : jsval_string(jsstr_empty()); *r = jsval_string(ret);
else else
jsstr_release(ret); jsstr_release(ret);
return S_OK; return S_OK;
......
...@@ -1395,6 +1395,8 @@ static HRESULT compile_continue_statement(compiler_ctx_t *ctx, branch_statement_ ...@@ -1395,6 +1395,8 @@ static HRESULT compile_continue_statement(compiler_ctx_t *ctx, branch_statement_
WARN("Label is not a loop\n"); WARN("Label is not a loop\n");
return JS_E_INVALID_CONTINUE; return JS_E_INVALID_CONTINUE;
} }
assert(pop_ctx != NULL);
}else { }else {
for(pop_ctx = ctx->stat_ctx; pop_ctx; pop_ctx = pop_ctx->next) { for(pop_ctx = ctx->stat_ctx; pop_ctx; pop_ctx = pop_ctx->next) {
if(pop_ctx->continue_label) if(pop_ctx->continue_label)
......
...@@ -454,6 +454,7 @@ static HRESULT prop_put(jsdisp_t *This, dispex_prop_t *prop, jsval_t val, IServi ...@@ -454,6 +454,7 @@ static HRESULT prop_put(jsdisp_t *This, dispex_prop_t *prop, jsval_t val, IServi
vdisp_release(&vthis); vdisp_release(&vthis);
return hres; return hres;
} }
/* fall through */
case PROP_PROTREF: case PROP_PROTREF:
prop->type = PROP_JSVAL; prop->type = PROP_JSVAL;
prop->flags = PROPF_ENUM; prop->flags = PROPF_ENUM;
......
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